Closed
[Documentation|文档反馈]: aclnnAllGatherMatmulV2 错误码表不完整、quantScale 支持口径与源码冲突(3处) #6789
liujing987913Add on  19 days agoClosed at  9 days ago
liujing987913
19 days ago Create

Document Link(文档链接)

涉及文档:aclnnAllGatherMatmulV2.md(master)

Issues Section(问题文档片段)

背景

通过自动化工具对现网 aclnn 文档(mc2/all_gather_matmul_v2/docs/aclnnAllGatherMatmulV2.md)与仓内源码进行比对。抓取的源码文件:mc2/all_gather_matmul_v2/op_api/aclnn_all_gather_matmul_v2.cpp、op_host/tiling 等。比对发现以下不一致,请确认。

问题明细

序号 文档现状 文档位置 源码依据 建议
1 161002 错误码行 rowspan=3,实际仅 1 行描述 错误码表·161002 段(第 381~385 行,rowspan 在 382~383,描述在 384) 表格自身格式错误(rowspan 与行数不符) 请修复 rowspan
2 161002 描述仅 1 条 错误码表·161002 行(第 384 行) 源码校验含 streamMode、format、commMode、shape、scale、amaxOut 等多个返回 161002 的场景 请确认错误码表补全
3 quantScale 参数使用说明:当前仅支持传入空指针场景。 参数表·quantScale 行(第 196~205 行,使用说明在第 200 行) aclnn 层支持非空 fp8/fp4 scalar 输入,tiling 层却要求 nullptr——两层口径冲突 请裁决以哪层为准,并统一文档

来源文件

Existing Issues(存在的问题)

详见问题片段

likedislike
周云飞
周云飞Collaborators
19 days ago Comments:

感谢您逐项整理文档与实现的差异。这三处需要分别确认表格格式、公开错误码和参数的实际支持范围,不能仅凭某一层的校验判断接口能力。

  1. 161002 行的表格格式:按您提供的片段,若该错误码下确实只有一行描述,rowspan=3 就与实际行数不符。可行的调整是先确定需要列出的错误场景,再将错误码及名称单元格的 rowspan 与最终描述行数保持一致,避免补充内容后仍存在跨行错位。

  2. 161002 的场景说明:您列出的 streamMode、format、commMode、shape、scale、amaxOut 校验是核对范围,但还需要区分内部校验失败与公开接口最终返回的错误码。对于确认会向调用者返回 161002 的分支,可以在错误码表中补充对应的触发条件;不能直接把所有内部失败分支都归入 161002。目前提供的源码片段尚未覆盖这些校验及返回路径,因此还不能确认完整的缺失清单。

  3. quantScale 的支持范围:不能简单裁定“以 aclnn 层为准”或“以 tiling 层为准”。即使 aclnn 层允许非空输入通过,也不代表对应执行路径能够完成后续处理;同样,某个 tiling 分支要求空指针,也不能直接推广到所有平台和模式。若您指出的两项检查处于同一执行路径,非空 quantScale 就可能在入口通过后被下游拒绝,此时不能据入口检查认定接口已支持该用法。需要结合平台、数据类型及分支条件确认端到端支持范围,再决定是保留“仅支持空指针”并统一入口限制,还是按实际可用条件补充非空输入说明。

目前的关键缺口是上述校验函数、调用分支和文档表格的完整内容,尚不足以确认非空 quantScale 的可用范围。继续确认时,重点应放在同一执行路径下的约束是否一致,以及对外返回码是否与文档对应。

likedislike
周云飞
周云飞Collaborators
19 days ago Comments:

/assign @w00951525

likedislike
CANN-robotCANN-robotCollaborators
19 days ago Assign w00951525 as responsible person
WwangfuhaoCollaborators
9 days ago Associated pull request
WwangfuhaoCollaborators
9 days ago Deleted associated pull request: 修复 aclnnAllGatherMatmul算子错误码表并统一 Full Mesh 组网表述
WwangfuhaoCollaborators
9 days ago Associated pull request
CANN-robotCANN-robotCollaborators
9 days ago Close issue
CANN-robotCANN-robotCollaborators
9 days ago changed custom state from 进行中 to 已完成
CANN-robotCANN-robotCollaborators
9 days ago Added label: resolved
周云飞
周云飞Collaborators
5 days ago Comments:

处置结果

已补齐 aclnnAllGatherMatmulV2 中英文文档的 161002 场景,涵盖 dtype、shape、模式、格式及量化相关指针约束,并对齐表格跨行。核对 tiling 的端到端约束后,quantScale 继续按仅支持空指针说明。

变更已合入 master,详见 PR #15461。

likedislike