执行摘要
- 一句话:新增 NPU VeOmni CI 流程与文档
- 推荐动作:值得合并,以提升 CI 覆盖。关注新添加测试脚本的稳定性,确保数据集预处理与训练流程能正确执行。
功能与动机
VeOmni 引擎在 NPU 上的 CI 测试覆盖不足,需要增加 e2e 训练和特定功能测试来保障代码质量。PR body 中提到"add two npu's veomni ci",直接点明动机。
实现拆解
- 新增 CI 步骤:在
.github/workflows/e2e_ppo_trainer_veomni_vllm_ascend.yml 中增加了三个新步骤:
- 预处理 GEO3K 数据集(
python examples/data_preprocess/geo3k.py)
- 运行 GEO3K E2E 训练测试(
bash tests/special_e2e/run_ppo_trainer_veomni.sh),使用 Qwen3-VL-2B-Instruct 模型
- 测试导出 unfused experts(
torchrun --nproc_per_node=8 tests/utils/veomni/test_special_export_unfused_experts.py)
- 更新文档:在
docs/ascend_tutorial/contribution_guide/ascend_ci_guide_zh.rst 的模型路径表格中添加了 5 个新模型条目:Qwen3-0.6B、Qwen3-8B-Base、Qwen3-VL-2B-Instruct、Qwen3-VL-4B-Instruct、Qwen3-4B-Instruct-2507,方便 CI 配置参考。
关键文件:
.github/workflows/e2e_ppo_trainer_veomni_vllm_ascend.yml(模块 CI;类别 infra;类型 infrastructure): 新增三个 CI 步骤(数据集预处理、E2E 训练、导出测试),是本次变更的核心。
docs/ascend_tutorial/contribution_guide/ascend_ci_guide_zh.rst(模块 文档;类别 docs;类型 documentation): 更新 CI 指南文档,新增 5 个 Qwen3 模型路径,便于 CI 配置参考。
关键符号:未识别
评论区精华
无人工 review 讨论。Gemini Code Assist bot 进行了自动评论,总结变更内容并宣布其即将停止服务。wucong25 直接批准了 PR。
风险与影响
- 风险:风险极低。新增的 CI 步骤在已有工作流文件中增加,不会影响现有步骤;文档更新仅为列表补充,不涉及逻辑变更。但需确保新添加的测试脚本和数据集路径在 NPU 环境中可用,否则可能导致 CI 失败。
- 影响:影响范围仅限于 NPU 环境的 CI 流程和中文文档。对用户功能和系统稳定性无直接影响,但增强了 VeOmni 引擎在 NPU 上的回归测试覆盖,有助于提前发现训练和导出相关问题。
- 风险标记:新 CI 步骤依赖数据集和脚本可用性
关联脉络
- PR #6667 [doc] feat: update Qwen3.5 NPU guide: 同为 NPU 文档更新,增加了 Qwen3.5 模型条目,与本 PR 的模型路径补充属于同一维护方向。
- PR #6637 [ci] test: add qwen3.5 2b ascend nightly ci: 添加 Qwen3.5 Ascend nightly CI,与本 PR 的 VeOmni CI 添加属于 CI 覆盖扩展的同类工作。
参与讨论