Closed
[Documentation|文档反馈]: 修正 StemOamPrepVarlenQ README 序列长度参数类型 #6101
KaranocaVeAdd on  Oct 13Closed at  9 days ago
KaranocaVe
KaranocaVe
Oct 13 Create

[Documentation|文档反馈]: 修正 StemOamPrepVarlenQ README 序列长度参数类型

问题

StemOamPrepVarlenQ README 将 qSeqLens 和 cuSeqLensQ 标为 Device 侧 aclTensor / INT32。实际接口使用 const aclIntArray*,元素为 INT64,Host 校验也要求 DT_INT64。按 README 构造参数会与接口不匹配并导致输入校验失败。

复核

对照 README 参数表与 ACLNN 声明、参数说明及 Host dtype 校验,可确认两项的类型和 dtype 标注不一致。

修复

将两项改为 aclIntArray / INT64,不改变算子接口或行为。

Validation

验证

  • ACLNN 声明将 qSeqLens、cuSeqLensQ 定义为 const aclIntArray*;API 示例以 int64_t 数据创建 aclIntArray。
  • Host OpDef 要求 ge::DT_INT64,Ascend 950 tiling 对非 INT64 输入返回失败。
  • README 两项现已标为 aclIntArray / INT64。
  • 补丁仅改参数表两行;git apply --check --whitespace=error-all 和 git diff --check 通过。
  • 目标仓库 pre-commit 4.3.0 对改动文件通过:trailing whitespace、EOF、large files、merge conflicts、private key、codespell、duplicate-header、OAT;不适用的 YAML/JSON、clang-format、ruff、compile-scope 检查跳过。
  • 未运行本地 CANN 构建或测试;由远端 CI 验证。
likedislike
周云飞
周云飞Collaborators
Oct 13 Comments:

/assign @KaranocaVe

likedislike
CANN-robotCANN-robotCollaborators
Oct 13 Assign KaranocaVe as responsible person
周云飞
周云飞Collaborators
29 days ago Comments:

感谢您指出这处参数说明不一致,并提供修订范围和检查结果。

按您提供的 ACLNN 声明及 Host 校验信息,qSeqLens、cuSeqLensQ 应以 const aclIntArray* 传入,元素类型为 INT64,而不是 Device 侧的 aclTensor / INT32。原标注会误导调用者构造不匹配的参数;其中,接口对象类型不匹配与 Host 对非 INT64 输入的校验失败属于不同层面的限制。

将 README 参数表中的这两项统一修正为 aclIntArray / INT64,并消除对应的 Device 侧描述,是与上述接口定义一致的调整方向。修订范围可限定在文档,不改变算子接口或计算行为。您已说明补丁通过静态检查、尚未运行本地 CANN 构建或测试;远端 CI 的验证结果仍待确认。

likedislike
周云飞周云飞Collaborators
13 days ago Associated pull request
CANN-robotCANN-robotCollaborators
9 days ago Close issue
CANN-robotCANN-robotCollaborators
9 days ago Added label: resolved
周云飞
周云飞Collaborators
5 days ago Comments:

已在 PR #15139 中修正 StemOamPrepVarlenQ 的参数说明:README 将 qSeqLens 和 cuSeqLensQ 改为 Host 侧 aclIntArray、INT64 类型,并分别以 batch 和 batch+1 描述数组长度;ACLNN 头文件中的对应注释也同步改为 INT64。

本次仅修改文档和注释,不改变接口或执行逻辑。

关联 PR:PR #15139

likedislike