Merged
【unfold_grad】修复issue问题 aclnnUnfoldGrad 文档内错误码正确分类 #6717
【unfold_grad】修复issue问题 aclnnUnfoldGrad 文档内错误码正确分类 #6717
Merged
tangkaidiAdd on 15 days ago
tangkaidiCollaborators
15 days ago

描述

aclnnUnfoldGrad 文档错误码表把 5 类约束统一归到 561002(ACLNN_ERR_INNER_TILING_ERROR),但其中至少 4 类在实际调用链上根本到不了 tiling。aclnnUnfoldGradGetWorkspaceSize 的 CheckParams 会先将其拦截并返回 161002(ACLNN_ERR_PARAM_INVALID)。
文档承诺的错误码与代码路径不符。

关联的Issue

[Documentation|文档反馈]: aclnnUnfoldGrad 文档把 4 类几何入参违规归入 561002,但 aclnn 层实际先返回 161002 #3827

测试

最新aclnnUnfoldGrad文档内错误码表格:
image.png
此外,不涉及影响精度的修改

文档更新

conversion/unfold_grad/docs/aclnnUnfoldGrad.md

类型标签

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for tangkaidi's contribution.)
TtangkaidiCollaborators
15 days ago Created pull request, commit faec0ae9
CANN-robotCANN-robotCollaborators
15 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
15 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
15 days ago Set FelixTang7,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
15 days ago Set gitee-duhuiping,gitcode-chenjiao,fengying555682,caiwenwen,yanglu-1 as approver
tangkaidiCollaborators
15 days ago Comments:

compile

likedislike
TtangkaidiCollaborators
15 days ago Pre merge succeeded (commit_id: 5af9a7c1c38df1311fc38cba5a9c84e01006e686)
CANN-robot
CANN-robotCollaborators
15 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
15 days ago Added label: ci-pipeline-running
6 event messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
15 days ago Removed label: ci-pipeline-passed
CANN-robot
CANN-robotCollaborators
15 days ago Comments:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robotCollaborators
15 days ago Added label: cann-cla/yes
5 event messages are collapsed here。 See more
TtangkaidiCollaborators
15 days ago Modify description of pull request
tangkaidiCollaborators
15 days ago Comments:

compile

likedislike
TtangkaidiCollaborators
15 days ago Pre merge succeeded (commit_id: f9faf33ab080affaa2d931e088af6150d878e938)
CANN-robot
CANN-robotCollaborators
15 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
15 days ago Added label: ci-pipeline-running
TtangkaidiCollaborators
15 days ago Modify description of pull request
CANN-robotCANN-robotCollaborators
15 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
15 days ago Added label: ci-pipeline-passed
chenjiaoCollaborators
15 days ago Comments:

/approve

likedislike
CANN-robotCANN-robotCollaborators
15 days ago Added label: approved
chenxingyu18Collaborators
15 days ago Comments:

/lgtm

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