Merged
【concat_d】修复issue问题 aclnnCat资料内第一段接口完成入参校验的错误码添加维度大于8一项 #6735
【concat_d】修复issue问题 aclnnCat资料内第一段接口完成入参校验的错误码添加维度大于8一项 #6735
Merged
tangkaidiAdd on 14 days ago
tangkaidiCollaborators
14 days ago

描述

aclnnCat 资料参数表写明 tensors 维度为 1-8,但第一段接口完成入参校验的错误码表没有维度大于 8这一项。
因此需要添加。

关联的Issue

[Documentation|文档反馈]: aclnnCat 参数表声明 1-8 维,错误码表漏列「维度大于8」(实测 rank=9 返回 161002) #3885

测试

添加维度大于8一项。
image.png

文档更新

conversion/concat_d/docs/aclnnCat.md

类型标签

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for tangkaidi's contribution.)
TtangkaidiCollaborators
14 days ago Created pull request, commit 798c5c38
atomgit-bot
atomgit-bot
14 days ago Comments:

变更摘要

本 PR 为 conversion/concat_d 算子的接口文档 aclnnCat.md 补充参数校验错误说明:在 aclnnCat 接口的 ACLNN_ERR_PARAM_INVALID(错误码 161002)错误项中新增一条约束,即 tensors 列表中 tensor 的维数大于 8 时返回该错误,并将该错误码对应的表格行合并范围由 5 行调整为 6 行。本次改动仅涉及文档描述,未涉及实现代码。

主要改动

  • 新增维数约束说明:在 aclnnCat 的 ACLNN_ERR_PARAM_INVALID / 161002 错误说明表格中新增一行,明确 tensors 列表中 tensor 的维数大于 8 时该接口返回参数无效错误。
  • 调整错误码表格合并范围:将 ACLNN_ERR_PARAM_INVALID 与对应错误码 161002 单元格的 rowspan 由 5 改为 6,使表格结构与新增的 1 行错误说明保持一致。

🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
atomgit-bot
atomgit-bot
14 days ago Comments:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
CANN-robotCANN-robotCollaborators
14 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
14 days ago Comments:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
*/*/docs/acl*.md ✅ 蔡雯雯, 汤平川 (2/2) ✅ 蔡雯雯 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

tangkaidi, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robotCollaborators
14 days ago Set yue-ma,zhou-qilong,gitee-duhuiping,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,gitcode-chenjiao,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,fengying555682,xuejinghui,onanfield,zhanw_coding,clinglai0517,caiwenwen,yanglu-1 as approver
CANN-robotCANN-robotCollaborators
14 days ago Set gitee-duhuiping,gitcode-chenjiao,fengying555682,caiwenwen,yanglu-1 as approver
tangkaidiCollaborators
14 days ago Comments:

compile

likedislike
TtangkaidiCollaborators
14 days ago Pre merge succeeded (commit_id: fffd144765cafe69e1ea892ae71ddd2017667e63)
CANN-robot
CANN-robotCollaborators
14 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
14 days ago Added label: ci-pipeline-running
8 event messages are collapsed here。 See more
TtangkaidiCollaborators
14 days ago Modify description of pull request
chenyifengCollaborators
14 days ago Comments:

您好,PR已满足合入指标,请尽快联系Committer进行合入。如暂时不需要合入,请在PR标题前加[WIP]

likedislike
TangPC
TangPCCollaborators
14 days ago Comments:

/lgtm
/approve

likedislike
caiwenwenCollaborators
14 days ago Comments:

/lgtm
/approve

likedislike
CANN-robot
CANN-robotCollaborators
14 days ago Comments:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 汤平川

likedislike
CANN-robotCANN-robotCollaborators
14 days ago Added label: lgtmapproved
CANN-robotCANN-robotCollaborators
14 days ago Closed associated issue
CANN-robotCANN-robotCollaborators
14 days ago Merged pull request