Prhub

#6169 [doc] fix: update dapo multi model optimization practice

原始 PR 作者 ChibiQuest 合并时间 2026-04-27 19:38 文件变更 1 提交数 2 评论 5 代码增减 +25 / -2

doc npu experimental

执行摘要

更新 DAPO 多模型优化实践文档

原文档中的启动方式不够直接，用户需要手动拼接命令或修改多处参数。更新后提供完整脚本，降低了上手门槛，适配 verl 0.7.1 的依赖要求。

建议精读 review 评论，尤其是 pkill -9 和路径硬编码问题，在后续文档更新中及时修复。对于用户而言，当前版本脚本需谨慎使用，建议手动替换 MASTER_ADDR 并确认 LD_PRELOAD 路径。

讨论亮点

Review 中 gemini-code-assist[bot] 提出了 4 条高优先级评论，主要关注：

  • pkill -9 python 过于暴力，可能影响其他用户进程，建议改为 pkill -u $USER -9 python 或依赖 ray 自身管理。
  • LD_PRELOAD 路径硬编码为 /usr/local/lib/libjemalloc.so.2，与文档前面 apt 安装路径（/usr/lib/aarch64-linux-gnu/ 或 /usr/lib/x86_64-linux-gnu/）不一致，会导致脚本失败。
  • 单节点脚本中 MASTER_ADDR 使用占位符，建议默认 127.0.0.1 以避免直接运行失败。
  • 多节点脚本中也存在同样的 pkill -9 python 风险。
    所有评论均未回复，PR 随后被 wucong25 批准并合并。

实现拆解

  1. 在 dapo_multi_model_optimization_practice.md 中，将单节点任务的说明从“修改脚本参数并执行”改为提供完整的 bash 启动脚本，包含环境变量配置、ray 启动和训练执行命令。
  2. 在多节点任务脚本中新增 VLLM_VERSION="0.13.0" 环境变量，并明确 NNODES、NPUS_PER_NODE、MASTER_ADDR 等参数的含义和修改要求。
  3. 删除原文档中“直接bash执行verl仓上示例脚本”的简单指引，替换为更结构化的脚本。
  4. 仅涉及单个文档文件，无代码或配置变更。
文件 模块 状态 重要度
docs/ascend_tutorial/examples/dapo_multi_model_optimization_practice.md 文档 modified 3.44

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

评论区精华

`pkill -9 python` 过于暴力 安全

gemini-code-assist[bot] 指出 `pkill -9 python` 会终止所有 Python 进程，建议改为 `pkill -u $USER -9 python` 或依赖 Ray 进程管理。

结论：未回应，PR 已合并，问题未修复。 · unresolved

LD_PRELOAD 路径与安装指南不一致 正确性

gemini-code-assist[bot] 指出 `/usr/local/lib/libjemalloc.so.2` 路径与 apt 安装路径不符，会导致脚本失败。

结论：未回应，PR 已合并，问题未修复。 · unresolved

MASTER_ADDR 无默认值 正确性

gemini-code-assist[bot] 建议单节点脚本默认使用 `127.0.0.1`，避免直接运行失败。

结论：未回应，PR 已合并，问题未修复。 · unresolved

风险与影响

文档中的启动脚本存在以下风险：

  1. pkill -9 python 在共享节点上可能误杀其他用户进程，导致数据丢失或服务中断。
  2. LD_PRELOAD 路径硬编码与文档前文安装路径不匹配，用户按脚本操作会失败。
  3. MASTER_ADDR 未提供默认值，直接复制运行会立即报错。
    这些风险主要影响首次使用文档的用户体验，但不会引发系统范围故障。

直接影响：修改了一个文档文件，仅影响阅读该教程的 Ascend 用户。
用户需要按新脚本拉起训练，但脚本中存在上述风险，可能导致首次运行失败。
团队需在后续更新中修复 review 指出的安全隐患和路径问题。

文档脚本可能因路径硬编码失败 `pkill -9 python` 存在安全风险 MASTER_ADDR 无默认值

关联 Issue

未识别关联 Issue

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

完整报告

Markdown PDF

参与讨论