Prhub

#37816 [CI/Build][LoRA] Update Qwen35 LoRA testing

原始 PR 作者 jeejeelee 合并时间 2026-03-23 12:55 文件变更 4 提交数 3 评论 1 代码增减 +369 / -135

执行摘要

更新 Qwen3.5 LoRA 测试,修复文件名 typo 并扩展视觉语言测试覆盖。

从 PR 标题“[CI/Build][LoRA] Update Qwen35 LoRA testing”和文件变更推断,动机是修复文件名 typo(如从 'densemoel' 改为 'densemodel')并扩展测试范围以包含视觉语言功能,提升测试的全面性和准确性。PR body 未提供具体表述,但基于标签和变更,这属于常规测试维护和优化。

该 PR 值得精读,尤其关注测试设计决策,如多模态测试的集成方式和夹具重构。建议工程师在合并后验证 fully_sharded_loras 测试覆盖情况,以确保 LoRA 功能完整性。

讨论亮点

Review 中仅有一次讨论,由 gemini-code-assist[bot] 提出:

  • 争议点:旧测试文件包含一个测试用例 test_qwen35_dense_model_lora_tp4_fully_sharded_loras,在新测试文件中缺失,可能导致 fully_sharded_loras 功能测试覆盖回归。
  • 结论:讨论未明确解决,但 reviewer Isotr0py 批准了 PR,暗示风险可接受或后续处理。
  • 未解决疑虑:未明确是否添加 dedicated 测试以覆盖 fully_sharded_loras 功能,这可能是一个潜在缺口。

实现拆解

实现方案包括四个文件变更:

  1. 修改 .buildkite/test_areas/lora.yaml:更新测试忽略列表,将 test_qwen35_densemoel_lora.py 替换为 test_qwen35_densemodel_lora.py,确保 CI 正确运行新测试。
  2. 修改 tests/lora/conftest.py:移除旧夹具 qwen35_dense_model_lora_files,新增 qwen35_text_lora_files 和 qwen35_vl_lora_files,分别用于文本和视觉语言 LoRA 测试。
  3. 新增 tests/lora/test_qwen35_densemodel_lora.py:包含新测试代码,支持文本 SQL 查询和视觉语言问答任务,使用 ImageAsset 处理图像,并定义辅助函数如 _run_text_lora_sample 和 _run_vl_lora_sample。
  4. 删除 tests/lora/test_qwen35_densemoel_lora.py:移除旧版测试文件,因其内容已被新文件覆盖。
文件 模块 状态 重要度
tests/lora/test_qwen35_densemodel_lora.py tests/lora added 6.0
.buildkite/test_areas/lora.yaml ci/build modified 4.0
tests/lora/conftest.py tests/lora modified 5.0

关键符号

_run_text_lora_sample _run_vl_lora_sample _assert_exact_outputs _assert_prefix_outputs

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

评论区精华

丢失的 fully_sharded_loras 测试覆盖 测试

gemini-code-assist[bot] 指出旧测试文件包含 test_qwen35_dense_model_lora_tp4_fully_sharded_loras 测试用例,在新文件中缺失,可能导致回归。

结论:未明确解决,但 PR 获批准,暗示风险可接受或待后续处理。 · unresolved

风险与影响

主要风险包括:

  1. 测试覆盖不全:移除旧测试文件可能丢失 fully_sharded_loras 测试用例(由 gemini-code-assist[bot] 指出),如果该功能重要,回归风险较高。
  2. 视觉语言功能引入新依赖:新测试使用 ImageAsset,如果图像处理逻辑或依赖不稳定,可能导致测试失败。
  3. 兼容性风险:CI 配置更新后,可能影响其他测试文件的并行执行,但变更范围小,风险较低。
    风险集中在测试层面,不影响生产代码核心功能。

影响分析如下:

  • 对用户:无直接影响,属于内部测试更新,不改变 API 或功能。
  • 对系统:提升 Qwen3.5 LoRA 测试的全面性,特别是视觉语言能力;CI 流程更准确,减少因文件名错误导致的测试失败。
  • 对团队:开发人员需适应新测试结构,但变更有限,学习成本低;可能需后续关注 fully_sharded_loras 测试覆盖问题。
    影响范围限于测试模块,程度为低到中。
缺少测试覆盖

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论