Prhub

#6298 [doc] refactor: ascend doc refactor of precision guide and dockerfile build guidance

原始 PR 作者 yyyy2000 合并时间 2026-05-12 16:52 文件变更 5 提交数 9 评论 4 代码增减 +204 / -11

执行摘要

重构 Ascend 精度对齐文档和 Dockerfile 构建指南

提供完整的 Ascend NPU 精度对齐指导,帮助用户在 NPU 上复现 GPU 训练精度并排查精度问题;更新 Dockerfile 构建文档以匹配最新依赖版本和最佳实践,降低用户搭建环境门槛。

建议 NPU 用户和文档维护者详细阅读精度对齐指南和 Dockerfile 构建指南,特别是容器启动模板的挂载路径和参数使用的准确性。该 PR 作为文档基础设施的改进,值得参考其组织方式。

讨论亮点
  • @gemini-code-assist[bot] 指出 Dockerfile 构建指南中挂载 /usr/local/sbin/usr/sbin 等主机系统目录进入容器会导致二进制不兼容,建议仅挂载 Ascend 驱动和固件路径。(dockerfile_build_guidance.rst 第 107 行)
  • @gemini-code-assist[bot] 指出精度对齐指南中使用 actor_rollout_ref.rollout.skip_dump_dir 参数疑似拼写错误,应使用 load_path,且末尾反斜杠可能引发 shell 语法错误。(precision_alignment_zh.md 第 104 行)
  • @wucong25 审核并批准了该 PR。

实现拆解

  1. 新增精度对齐指南precision_alignment_zh.md):从环境与权重对齐、输入数据对齐、配置对齐、固定确定性、训练验证、推理验证到 dump 对比,完整覆盖精度对齐工作流。详细阐述了 HCCL/LCCL 通信下确定性环境变量设置、rollout 打桩和回放验证方法。
  2. 迁移并更新 Dockerfile 构建指南dockerfile_build_guidance.rst):从 quick_start 目录移至 get_start 目录,更新依赖版本表(vLLM->0.18.0, Torch->2.9.0, SGLang->0.5.10 等),新增容器启动命令模板、容器管理说明(start/exec),并强调镜像为参考样例。
  3. 更新文档索引index.rst):添加新增的精度对齐文档路径,调整 dockerfile 构建指南路径,删除旧的 precision_debugger.md 索引(已移至新位置)。
  4. 配套 CI 脚本更新run_gspo_qwen3_30b_megatron_npu.sh):在 rollout 配置中增加 calculate_log_probs=True 参数,确保在 GSPO 训练中正确计算 log probability。
文件 模块 状态 重要度
docs/ascend_tutorial/dev_guide/precision_analysis/precision_alignment_zh.md 精度指南 added 5.24
docs/ascend_tutorial/get_start/dockerfile_build_guidance.rst 构建指南 renamed 4.59
tests/special_npu/nightly_ci_ascend/run_gspo_qwen3_30b_megatron_npu.sh 测试脚本 modified 3.28
docs/index.rst 文档索引 modified 1.96
docs/ascend_tutorial/dev_guide/precision_analysis/precision_debugger_zh.md 调试工具 renamed 1.35

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

评论区精华

容器挂载系统目录风险 安全

Review 指出挂载 /usr/local/sbin 和 /usr/sbin 可能导致二进制不兼容,建议仅挂载 Ascend 驱动和固件路径。

结论:PR 合并时未修改,建议后续文档更新中修正或增加警告。 · unresolved

参数 skip_dump_dir 可能拼写错误 正确性

Review 指出 actor_rollout_ref.rollout.skip_dump_dir 应为 load_path,且末尾反斜杠可能引发 shell 语法错误。

结论:PR 合并时保持原状,用户需注意实际参数名称。 · unresolved

风险与影响

文档中使用的参数 skip_dump_dir 可能是拼写错误,建议用户根据实际版本使用 load_path;容器启动模板中挂载系统目录存在安全隐患和兼容性风险,应在后续文档中更正或新增警告。测试脚本新增一行配置,影响 CI 的正确性,但经过验证无误。

对 Ascend NPU 用户提供了一站式精度对齐指南,降低排查难度;Dockerfile 构建指南更新使得用户更容易获取最新环境;文档结构调整提升了可维护性。

参数错误风险 容器挂载风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论