Prhub

#22704 [NPU] --attn-cp-size --init-expert-location --eplb-algorithm parameter docs update

原始 PR 作者 Hide-on-bushsh 合并时间 2026-04-13 22:42 文件变更 1 提交数 6 评论 2 代码增减 +3 / -3

执行摘要

更新 Ascend NPU 文档,澄清三个参数的约束和有效值。

根据PR标题和body中的截图(未提供具体内容,但标题表明是参数文档更新),动机是更新Ascend NPU相关参数的文档,以澄清其约束和有效值,确保用户正确配置。这符合近期多个NPU文档PR(如#22705、#22687、#22700)的维护趋势,旨在持续改进平台文档的准确性和可用性。

该PR是简单的文档更新,无需精读。对于关注Ascend NPU平台或文档维护的工程师,可快速浏览以了解参数约束的澄清;对于其他开发者,可直接忽略。

讨论亮点

review中只有gemini-code-assist[bot]提出了两条格式建议:

  1. 建议--attn-cp-size的约束描述使用分号而非括号以保持与文档其他部分(如--pp-size)的一致性;
  2. 指出--eplb-algorithm的选项列表中不应使用尖括号(如<deepseek>),因为deepseek是字面值而非占位符,避免用户混淆。作者在后续提交中采纳了这些建议,sglang-npu-bot最终批准了PR。没有涉及技术设计或正确性的深度讨论。

实现拆解

仅修改了一个文件docs/platforms/ascend/ascend_npu_support_features.md,包含三处关键改动:

  1. --attn-cp-size参数在“Type”列添加约束“must be equal to --tp-size”;
  2. --init-expert-location参数在“Type”列将“Type: str”替换为具体有效值列表“trivial, , , ”;
  3. --eplb-algorithm参数在“Type”列将“Type: str”替换为选项“auto, deepseek”。
文件 模块 状态 重要度
docs/platforms/ascend/ascend_npu_support_features.md documentation modified 3.0

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

评论区精华

文档格式一致性 style

gemini-code-assist[bot] 建议在 `--attn-cp-size` 的约束描述中使用分号而非括号,以与文档中其他参数(如 `--pp-size`)的格式保持一致。

结论:作者采纳建议,在后续提交中更新了格式。 · 已解决

字面值与占位符区分 documentation

gemini-code-assist[bot] 指出 `--eplb-algorithm` 的选项列表中不应使用尖括号(如 `<deepseek>`),因为 `deepseek` 是字面值,尖括号通常表示占位符,可能引起用户混淆。

结论:作者采纳建议,移除了尖括号。 · 已解决

风险与影响

风险极低:

  1. 纯文档变更,不涉及代码逻辑,无回归风险;
  2. 修改内容为澄清约束和选项,若原文档有误则可能纠正潜在的用户配置错误,但无直接性能或安全影响;
  3. 唯一风险是文档准确性,但review中已通过格式建议确保一致性,且变更基于已知参数行为。

影响范围有限:

  1. 对用户:帮助Ascend NPU用户更清晰地理解--attn-cp-size--init-expert-location--eplb-algorithm参数的用法,减少配置错误,提升体验;
  2. 对系统:无运行时影响;
  3. 对团队:延续了近期NPU文档维护的节奏(如#22705、#22687等),属于常规文档更新。
纯文档变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论