Prhub

#6272 [doc] fix: router_replay is now under megatron

原始 PR 作者 HollowMan6 合并时间 2026-05-08 15:21 文件变更 1 提交数 1 评论 4 代码增减 +13 / -9

执行摘要

router_replay 文档配置键路径修正

PR 描述指出:router_replay configuration key has been moved under megatron key, but the documentation hasn't reflected this correctly. 因此需要同步更新文档以避免用户使用错误的配置路径。

该 PR 属于常规文档维护,改动量小且正确性明确,无需深度阅读。但可以注意:文档与代码结构同步更新是良好的实践。

讨论亮点

Review 中 gemini-code-assist[bot] 指出 R3 模式的 YAML 示例中 enable_rollout_routing_replay 缺少父级 rollout 键,可能导致配置错误。Copilot 建议 YAML 示例应从 actor_rollout_ref 根级开始以与 CLI 示例一致,但作者 HollowMan6 回复 “adjusted” 后未进一步调整。最终 reviewer wuxibin89 批准了 PR。

实现拆解

  1. 修正 R2 模式 YAML 示例:在 examples/router_replay/README.md 中,将原来顶层的 router_replay 移至 actor.megatron 下,使 YAML 片段与实际配置结构一致。
  2. 修正 R2 模式 CLI 示例:将 actor_rollout_ref.actor.router_replay.mode 改为 actor_rollout_ref.actor.megatron.router_replay.mode
  3. 修正 R3 模式 YAML 示例:将 actor 部分的 router_replay 嵌套到 actor.megatron 下,同时补全 rollout 部分的父级键 rollout:,确保 enable_rollout_routing_replay 正确放置在 rollout 下。
  4. 修正 R3 模式 CLI 示例:将 actor_rollout_ref.actor.router_replay.mode 改为 actor_rollout_ref.actor.megatron.router_replay.mode
文件 模块 状态 重要度
examples/router_replay/README.md 文档 modified 2.81

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

评论区精华

R3 模式 YAML 示例缺少 rollout 父级键 正确性

gemini-code-assist[bot] 指出 `enable_rollout_routing_replay` 缺少 `rollout` 父级键,可能导致配置错误。

结论:作者 HollowMan6 回复 'adjusted',PR 最终合并时已修正该问题。 · 已解决

YAML 示例根级不完整 设计

Copilot 建议 YAML 示例包含 `actor_rollout_ref` 根级以与 CLI 示例一致,但作者仅回复 'adjusted' 并未调整。

结论:未完全采纳,PR 仍合并。可能认为当前示例对于熟悉配置的用户已足够清晰。 · unresolved

风险与影响

此 PR 仅为文档变更,不涉及代码逻辑,因此没有回归、性能、安全或兼容性风险。但若文档示例仍存在歧义(如 YAML 根级不完整),用户可能仍会配置错误。

影响范围:仅影响阅读 examples/router_replay/README.md 的用户。影响程度:低。用户若使用过时的配置路径会遇到错误,更新文档后能正确配置。

缺少父级键曾被遗漏 文档示例根级不完整

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论