Prhub

#38429 [CI] Fix Ernie4.5-VL initialization test

原始 PR 作者 haosdent 合并时间 2026-03-28 22:43 文件变更 1 提交数 1 评论 2 代码增减 +1 / -1

执行摘要

更新 Ernie4.5-VL 模型测试的 HF revision 至 refs/pr/17,修复 CI 初始化测试失败。

PR body指出上游HF repo在PR #17中修复了两个bug:

  1. Ernie4_5_MoEConfig.__init__访问self.num_hidden_layers前未由父类设置,导致to_diff_dict()崩溃;
  2. processing_ernie4_5_vl.py中的顶层import decord在没有安装decord时阻塞tokenizer加载。这些bug导致vLLM的测试失败,因此需要更新revision以使用修复后的版本。

此PR值得快速浏览以了解CI依赖管理实践,重点关注gemini-code-assist[bot]的建议(尽管未采纳),它提醒在测试中应优先使用commit hash以确保长期稳定性和可复现性。

讨论亮点

review中gemini-code-assist[bot]提出:'For improved test stability and reproducibility, it is best practice to pin the revision to a specific commit hash rather than a pull request reference.' 建议使用commit hash而非PR引用以避免未来测试失败。但该建议未被采纳,PR最终使用PR引用,讨论揭示了CI测试中依赖可变引用 vs 固定hash的权衡。

实现拆解

在tests/models/registry.py文件的check_available_online函数中,将Ernie4_5_VLMoeForConditionalGeneration的_HfExamplesInfo配置的revision字段从'reds/pr/14'改为'reds/pr/17',无其他代码或文件改动。

文件 模块 状态 重要度
tests/models/registry.py 测试 / 模型注册 modified 3.0

关键符号

check_available_online

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

Revision pinning strategy for CI tests 设计

gemini-code-assist[bot] 建议使用 commit hash 以提高测试稳定性和可复现性,反对使用可变 PR 引用如 'reds/pr/17'。

结论:建议未被采纳,PR 保持使用 PR 引用,讨论后未修改。 · 已解决

风险与影响

主要风险是使用PR引用(如'reds/pr/17')而非commit hash可能导致未来测试不稳定,如果PR引用被移除或更改,可能再次引发CI失败。但短期风险低,因为上游bug已修复;无回归、性能、安全或兼容性影响,仅修改测试配置。

影响范围限于CI测试系统,确保Ernie4.5-VL多模态模型的初始化测试通过,对最终用户和核心系统无直接影响。影响程度低,属于常规维护,旨在保持CI稳定性。

潜在测试不稳定 依赖外部 PR 引用

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论