Prhub

#6667 [doc] feat: update Qwen3.5 NPU guide

原始 PR 作者 zjchenn 合并时间 2026-06-09 16:37 文件变更 4 提交数 3 评论 0 代码增减 +137 / -91

执行摘要

统一 Qwen3.5 NPU 文档,支持 35B 和 122B

PR 标题和描述指出需要对 Qwen3.5 的 Ascend 文档进行更新,以覆盖 35B 和 122B 两种模型尺寸。原有的文档仅针对 122B 模型,新增 35B 支持后需要合并为一份指南,便于用户查阅并统一镜像和依赖版本。

建议阅读新建的统一指南 qwen3_5_megatron_npu.md,了解 Qwen3.5 在 Ascend NPU 上使用 Megatron+vLLM 进行 GRPO 训练的完整流程,特别是多机 Ray 集群启动参数和并行策略配置。维护者应关注 Docker 镜像和 commit 的时效性,及时更新文档中的版本号。

讨论亮点

该 PR 无实质性人工讨论。合并前仅有 Gemini Code Assist 机器人自动评论总结变更内容,随后被维护者 wucong25 直接批准合并,未提出任何修改意见。

实现拆解

实现步骤如下:

  1. 创建统一指南:在 docs/ascend_tutorial/model_support/examples/ 下新增 qwen3_5_megatron_npu.md,同时包含 Qwen3.5-35B-A3B 和 Qwen3.5-122B-A10B 两个模型的版本要求、硬件并行配置(TP/PP/CP/EP/ETP/GEN_TP)、数据准备命令、Ray 集群启动步骤以及训练执行示例。
  2. 删除旧文件:移除原有的 qwen3_5_122b_npu.md,避免内容重复。
  3. 更新引用:修改 model_and_algorithm_support.md 中第 41 行(Qwen3.5-122B-A10B)的示例链接,从旧文件指向新文件的路径。
  4. 调整索引:更新 docs/ascend_tutorial/index.rst 中的 toctree 条目,将 qwen3_5_122b_npu 替换为 qwen3_5_megatron_npu
    所有变更均为纯文档操作,不涉及任何代码逻辑修改。
文件 模块 状态 重要度
docs/ascend_tutorial/model_support/examples/qwen3_5_megatron_npu.md 文档 A 4.47
docs/ascend_tutorial/model_support/examples/qwen3_5_122b_npu.md 文档 D 3.92
docs/ascend_tutorial/model_support/model_and_algorithm_support.md 文档 M 1.93
docs/ascend_tutorial/index.rst 文档 M 1.72

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

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

虽然是文档变更,但仍有以下风险需关注:

  1. 链接断裂:旧文件 qwen3_5_122b_npu.md 被删除,若其他文档或外部 URL 直接引用该路径,将导致 404。需确认仓库内及外部引用均已更新。
  2. 版本同步:新文件推荐的 Docker 镜像 Tag verl-9.0.0-a3-ubuntu22.04-py3.11-latest 和 CANN 版本 9.0.0 必须与镜像构建实际情况一致,否则用户可能无法使用。
  3. Commit 引用:文档中指定 verl commit 为 cdd9014f,该 Commit 可能随开发推进而被新 Commit 替代,用户需要关注对应版本的兼容性。

对用户:NPU 场景用户将获得更完整的 Qwen3.5 GRPO 训练文档,特别是新增了 35B 模型的示例配置;需要从旧文档迁移的用户需注意路径变化。对团队:文档维护成本降低,后续新增模型只需修改统一指南。影响面仅限于文档内容,无功能影响。

文档链接断裂风险 依赖版本可能过期 删除文件后外部引用需检查

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论