Prhub
← 返回仓库详情

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

sgl-project/sglang · 作者 Hide-on-bushsh · 合并时间 2026-04-13 22:42

分析状态 已生成
文件变更 1提交数 6 · 评论 2
代码增减 +3 / -3
documentation npu

执行摘要

更新 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 链接，后续同步到相关引用后会出现在这里。

完整报告

Markdown PDF

执行摘要

本次PR更新了Ascend NPU支持特性文档，澄清了--attn-cp-size、--init-expert-location和--eplb-algorithm三个参数的约束和有效值，属于低风险、小范围的文档维护，旨在提升用户配置的清晰度，无需深入技术审查。

功能与动机

根据PR标题和body（含截图，但内容未提供），动机是更新Ascend NPU相关参数的文档，以澄清其约束和有效值。这延续了近期多个NPU文档PR（如#22705、#22687、#22700）的维护趋势，旨在持续改进平台文档的准确性和可用性，帮助用户避免配置错误。

实现拆解

仅修改了docs/platforms/ascend/ascend_npu_support_features.md文件，具体变更如下：
| 参数 | 变更前 | 变更后 | 说明 |
|------|--------|--------|------|
| --attn-cp-size | Type: int | Type: int; must be equal to --tp-size | 添加了必须等于--tp-size的约束 |
| --init-expert-location | Type: str | trivial, <path.pt>, <path.json>, <json_string> | 明确了有效值格式，包括字面值和文件路径占位符 |
| --eplb-algorithm | Type: str | auto, deepseek | 修正为具体选项列表，移除了可能引起混淆的尖括号 |

评论区精华

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

"For consistency with the formatting in line 89, consider using a semicolon instead of parentheses to describe the constraint."
"The use of angle brackets (<...>) typically denotes a placeholder for a user-provided value. Since deepseek is a literal value for this parameter, the angle brackets should be removed to avoid confusion."
作者在后续提交中采纳了这些建议，确保了文档格式的一致性和清晰性。

风险与影响

  • 风险：极低，纯文档变更无代码逻辑影响；唯一潜在风险是文档准确性，但review已纠正格式问题。
  • 影响：帮助Ascend NPU用户更清晰地理解参数用法，减少配置错误；对系统无运行时影响；属于常规文档维护，延续了近期NPU文档更新节奏。

关联脉络

本次PR是近期Ascend NPU文档维护序列的一部分，与#22705、#22700、#22697等PR类似，都修改了ascend_npu_support_features.md文件，反映了团队对NPU平台文档持续优化的重视。这些变更共同提升了文档的完整性和用户友好性，支持SGLang在异构硬件（如Ascend NPU）上的部署体验。

参与讨论