Prhub

#6573 [doc] chore: Update ascend_sglang_best_practices.rst

原始 PR 作者 hustmf 合并时间 2026-06-02 19:23 文件变更 1 提交数 2 评论 2 代码增减 +13 / -31

执行摘要

更新 SGLang 最佳实践文档

文档过期,部分模型脚本路径变更,需要同步更新以保持文档准确性。

该 PR 为常规文档维护,合并后没有风险,无需特别关注。

讨论亮点

Gemini Code Assist 机器人发现两个问题:

  1. 文档第 97 行仍引用旧的 Qwen3-30B 脚本路径,与更新后的路径不一致。
  2. 删除 Qwen2.5-32B 后,第 175 行的交叉引用会失效,导致 Sphinx 编译失败。
    这两个问题在最终合并前已修复。

实现拆解

  1. 更新文档更新时间戳为 06/02/2026。
  2. 删除 Qwen2.5-32B 模型链接及相关内容。
  3. 将 Qwen3-30B 脚本路径更新为 examples/ascend_extras/grpo_trainer/run_qwen3_30b_a3b_megatron.sh。
  4. 在性能采集部分添加新的 pip 依赖项。
  5. 更新文档内部引用链接,修复因删除 Qwen2.5-32B 导致的断链。
文件 模块 状态 重要度
docs/ascend_tutorial/model_support/examples/ascend_sglang_best_practices.rst 文档 modified 3.04

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

评论区精华

旧脚本路径未更新 正确性

Gemini Code Assist 指出第 97 行仍引用旧的 Qwen3-30B 脚本路径 `examples/grpo_trainer/run_qwen3moe-30b_sglang_megatron_npu.sh`,与更新后的路径不一致。

结论:已修复,路径统一更新。 · 已解决

Qwen2.5-32B 删除导致断链 正确性

Gemini Code Assist 指出删除 Qwen2.5-32B 后,第 175 行的交叉引用 `` `Qwen2.5-32B`_ `` 成为无效引用,会导致 Sphinx 编译失败。

结论:已修复,移除了该引用或替换为通用描述。 · 已解决

风险与影响

低风险。仅涉及文档内容的更新与修复,不影响代码逻辑。未修复的旧脚本路径引用可能导致用户执行错误的脚本。

影响范围限定于 Ascend SGLang 最佳实践文档的用户。更新后文档更准确,避免了无效链接和错误的脚本路径。

文档引用不一致(已修复)

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论