Closed
[Documentation|文档反馈]: attention_update: README 将 A2/A3 的 sp 上限误写为 16 #6955
aoye_poppyAdd on  16 days agoClosed at  10 days ago
aoye_poppy
16 days ago Create

attention/attention_update/README.md:80

源码提交:72e64e29ed30a8a035871083055dc5645dfc2518。

Issues Section / 问题文档片段

README 的约束说明统一写为:

序列并行的并行度sp取值范围[1, 16]。

Existing Issues / 存在的问题

README 同时声明支持 Atlas A2、Atlas A3 和 Ascend 950PR&950DT 产品,但不同产品的 sp 上限不同:

  • Atlas A2/A3:sp 范围应为 [1,128]。
  • Ascend 950PR&950DT:sp 范围为 [1,16]。

统一写 [1,16] 会使用户误判 A2/A3 的 sp=17..128 合法调用不受支持。

交叉证据:

  1. API 文档第 264 行 明确 A2/A3 为 [1,128],第 267 行明确 950 为 [1,16]。
  2. API 实现第 47-48 行 分别定义 SP_MAX=128、SP_MAX_95=16;CheckSp 与 CheckSp_95 使用对应上限。

【确认】已交叉核对 README、API 文档和分平台校验实现;本条为产品约束范围说明不一致。

Suggested correction / 建议修改

将原句拆为按产品描述:

Atlas A2系列产品、Atlas A3系列产品:序列并行的并行度sp取值范围[1,128]。
Ascend 950PR&950DT系列产品:序列并行的并行度sp取值范围[1,16]。
likedislike
周云飞
周云飞Collaborators
16 days ago Comments:

感谢您提供详细的文档位置和交叉对照信息。

您报告的关键问题是 README 未区分不同产品的 sp 上限。实现中的 SP_MAX=128 和 SP_MAX_95=16 也表明上限并非统一为 16。结合您提供的 API 文档说明,README 统一写为 [1,16] 会让用户误以为 A2/A3 的 sp=17..128 超出支持范围。

建议将 README 中这条约束按产品拆分为:

  • Atlas A2 系列产品、Atlas A3 系列产品:序列并行的并行度 sp 取值范围为 [1,128]。
  • Ascend 950PR&950DT 系列产品:序列并行的并行度 sp 取值范围为 [1,16]。

这一调整方向是让文档准确表达不同产品的已有约束,而不是将所有产品的上限统一放宽到 128。本条可按文档约束说明不一致的问题继续确认,无需您补充复现环境。

likedislike
周云飞周云飞Collaborators
15 days ago Assign L_Euler as responsible person
CANN-robotCANN-robotCollaborators
10 days ago Close issue
CANN-robotCANN-robotCollaborators
10 days ago Added label: resolved
yyt_hwyyt_hwCollaborators
10 days ago changed custom state from 进行中 to 已完成
HHan-XiaobinCollaborators
10 days ago Assign yyt_hw as responsible person
周云飞
周云飞Collaborators
5 days ago Comments:

PR #14910 更新了 README,并在 DecodeUpdate 的 tiling 中增加了 sp 范围为 1~128 及输入数量的校验。README 中 A2/A3 的 sp 上限是否已更正,尚不能确认。

关联 PR:PR #14910

likedislike