Prhub

#6253 [ci] fix: remove the rebundant config

原始 PR 作者 yyyy2000 合并时间 2026-05-07 14:05 文件变更 1 提交数 1 评论 1 代码增减 +1 / -2

执行摘要

移除 NPU 测试脚本中冗余的 context_parallel_size 配置

PR 标题说明为 'remove the rebundant config',旨在清理 NPU 夜间 CI 脚本中的冗余配置,避免潜在的配置冲突或阅读歧义。来自 gemini-code-assist[bot] 的 review 也确认该去除是正确的。

建议快速合并。这是低风险的清理变更,已经过维护者批准。对于 review 中提出的 actor 侧冗余和缺失配置问题,可以另开 PR 跟进,但当前 PR 的 scope 限定在 ref 模型冗余配置清理是合理的。

讨论亮点

来自 gemini-code-assist[bot] 的 review 评论指出:actor 模型配置中也存在类似的冗余(第122行的 tensor_model_parallel_size 和第136行的 context_parallel_size),建议一并清理以保持一致性。同时指出参考模型配置缺少 use_mbridge=Truemulti_latent_attention 等重要设置。

审阅者 wucong25 最终给予了 APPROVED,表明核心问题已被接受,冗余清理符合预期。不过机器人建议的 actor 侧冗余清理和缺失配置补充未被采纳,可能留待后续 PR 处理。

实现拆解

  1. 移除冗余的 override 配置:在 tests/special_npu/nightly_ci_ascend/run_dapo_moonlight-16b_megatron_npu.sh 脚本的第172行,删除 ++actor_rollout_ref.ref.megatron.override_transformer_config.context_parallel_size=${REF_CP} 这一行。该行与上一行的 actor_rollout_ref.ref.megatron.context_parallel_size=${REF_CP} 功能重复,因为在 Megatron 中通过 override_transformer_config 设置再写一遍 context_parallel_size 是冗余的。
  2. 修复文件末尾换行符:在脚本最后一行末尾添加上换行符,修复 No newline at end of file 问题,保持文本文件规范。
  3. 仅涉及测试配置:变更文件只有一个,属于测试用例的配置脚本,不影响任何源码逻辑、API 或运行时行为。
文件 模块 状态 重要度
tests/special_npu/nightly_ci_ascend/run_dapo_moonlight-16b_megatron_npu.sh NPU 测试 modified 3.12

关键源码片段

tests/special_npu/nightly_ci_ascend/run_dapo_moonlight-16b_megatron_npu.sh test-coverage

唯一变更文件,移除参考模型冗余的 context_parallel_size override 配置,并修复末尾换行。

# 移除前后的关键对比 :
# 删除前 (base):
actor_rollout_ref.ref.megatron.context_parallel_size=${REF_CP} \
++actor_rollout_ref.ref.megatron.override_transformer_config.context_parallel_size=${REF_CP} # 冗余行# 删除后 (head):
actor_rollout_ref.ref.megatron.context_parallel_size=${REF_CP} \  # 直接设置已足够# 文件末尾修复 : 添加了换行符,避免 'No newline at end of file' 警告

评论区精华

建议一并清理 actor 配置中的类似冗余并补充缺失设置 设计

gemini-code-assist[bot] 建议:actor 配置中也有相似的冗余 override(tensor_model_parallel_size 和 context_parallel_size),应同步清理;此外参考模型配置缺少 use_mbridge=True 和 multi_latent_attention 等关键设置,应补充。

结论:审阅者 wucong25 批准了 PR,表明当前改动已足够;机器人建议可能留待后续 PR 处理。 · 已解决

风险与影响

风险极低。变更仅删除了一条逻辑上完全冗余的配置行,该配置与上一行功能等效,删除后不会影响实际行为。唯一可能的风险是,若该冗余配置在某些特定分支或内部版本中被用作特殊覆盖入口,则删除可能改变行为,但在上游主线中可能性很小。文件末尾换行符修复无风险。

影响范围极小。仅影响 NPU 晨间/夜间 CI 的 DAPO Moonlight-16B Megatron 测试脚本,且仅为配置清理,不影响任何用户功能、训练结果或系统稳定性。对于开发者和测试者,脚本可读性略微提升。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论