# PR #6253 完整报告

- 仓库：`verl-project/verl`
- 标题：[ci] fix: remove the rebundant config
- 合并时间：2026-05-07 14:05
- 原文链接：http://prhub.com.cn/verl-project/verl/pull/6253

---

# 执行摘要

- 一句话：移除 NPU 测试脚本中冗余的 context_parallel_size 配置
- 推荐动作：建议快速合并。这是低风险的清理变更，已经过维护者批准。对于 review 中提出的 actor 侧冗余和缺失配置问题，可以另开 PR 跟进，但当前 PR 的 scope 限定在 ref 模型冗余配置清理是合理的。

# 功能与动机

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

# 实现拆解

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 测试；类别 test；类型 test-coverage）: 唯一变更文件，移除参考模型冗余的 context_parallel_size override 配置，并修复末尾换行。

关键符号：未识别

## 关键源码片段

### `tests/special_npu/nightly_ci_ascend/run_dapo_moonlight-16b_megatron_npu.sh`

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

```bash
# 移除前后的关键对比 :
# 删除前 (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' 警告

```

# 评论区精华

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

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

- 建议一并清理 actor 配置中的类似冗余并补充缺失设置 (design): 审阅者 wucong25 批准了 PR，表明当前改动已足够；机器人建议可能留待后续 PR 处理。

# 风险与影响

- 风险：**风险极低**。变更仅删除了一条逻辑上完全冗余的配置行，该配置与上一行功能等效，删除后不会影响实际行为。唯一可能的风险是，若该冗余配置在某些特定分支或内部版本中被用作特殊覆盖入口，则删除可能改变行为，但在上游主线中可能性很小。文件末尾换行符修复无风险。
- 影响：**影响范围极小**。仅影响 NPU 晨间 / 夜间 CI 的 DAPO Moonlight-16B Megatron 测试脚本，且仅为配置清理，不影响任何用户功能、训练结果或系统稳定性。对于开发者和测试者，脚本可读性略微提升。
- 风险标记：暂无

# 关联脉络

- 暂无明显关联 PR