Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhaozhoujun520 的贡献)变更摘要
本PR主要修正了 aclnnConvolutionBackward 及相关反向传播算子(ConvTbcBackward、DeformableConv2dBackward)在异常场景下的错误提示信息,包括修复语法错误、统一标点符号、补充缺失的参数信息以及更正不准确的描述文字,使报错信息更加规范和准确。
主要改动
-
修正
CheckDeterministic中 deterministic 计算的 groups 约束描述:在aclnn_convolution_backward.cpp中,将错误信息从 "the value of groups must be less than or equal to 1" 改为 "the value of groups must be equal to 1",使 deterministic 计算场景下的约束描述更加精确。 -
修复
convolution_backward_checker.cpp中的报错信息语法错误:修正了CheckDtypeValidForBpFilter8bit中重复的 "is is" 为正确的 "is",并统一了CheckConvChannelAndGroup中 channel 维度检查报错信息的大小写及介词使用("must equal to" → "must be equal to")。 -
修复
convtbc_backward_checker.cpp中 pad 检查的报错措辞:将 "the value of pad must be greater or equal to 0" 改为 "the value of pad must be greater than or equal to 0",补充了缺失的 "than"。 -
规范
deformable_conv2d_backward_checker.cpp中的报错信息格式:将 stride 和 dilation 检查信息中的中文顿号「、」统一替换为英文逗号;在CheckShape中为整除检查的报错信息补充了groups和deformableGroups参数的实际值,并将 "all axis" 改为 "All axis" 以统一首字母大写风格。


代码审查
所有 4 个文件都已审查完毕。本次 diff 仅涉及错误消息文本的修正(语法修复、标点符号统一、增加缺失的诊断参数值),未引入任何逻辑错误、安全风险、可靠性问题或破坏性变更。
具体审查结果:
-
aclnn_convolution_backward.cpp — 错误消息从 "must be less than or equal to 1" 修正为 "must be equal to 1",与条件逻辑
groups > 1一致。无问题。 -
convolution_backward_checker.cpp — 第 404 行修复重复词 "is is"→"is";第 646 行修正语法 "must equal to"→"must be equal to"、大写首字母并移除多余句号。无问题。
-
convtbc_backward_checker.cpp — 第 100 行补全缺失的 "than":"greater or equal"→"greater than or equal"。无问题。
-
deformable_conv2d_backward_checker.cpp — 第 160/170 行将中文顿号(、)统一为英文逗号(,);第 214-216 行在 deformableGroups 整除检查的错误诊断中增加 deformableGroups 值;第 218-220 行在 groups 整除检查的错误诊断中增加 groups 值并大写 "All"。无问题。
关闭总结
- 各优先级发现数:P0: 0, P1: 0, P2: 0, P3: 0
- 整体风险评估:无风险。本变更为纯错误消息文本修正,不涉及任何运行时行为变更。
已审查的 4 个文件:
conv/convolution_backward/op_api/aclnn_convolution_backward.cpp— 无问题conv/convolution_backward/op_api/convolution_backward_checker.cpp— 无问题conv/convolution_backward/op_api/convtbc_backward_checker.cpp— 无问题conv/convolution_backward/op_api/deformable_conv2d_backward_checker.cpp— 无问题
⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。


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
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| ** | ✅ 杨阳, gcw_kUomxQ2l (2/2) | ✅ gcw_kUomxQ2l (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
zhaozhoujun520, thanks for your pull request. All authors of the commits have signed the CLA. 👍


/compile


流水线任务触发成功
任务链接 [44c49e3d7bea4ebab567b1c1718f539c][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| Compile_Ascend_X86_mobile_station | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_single | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_experimental | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Pre | ✅ SUCCESS | >>>>> | |
| pre_comment | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910b | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910c | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_harmony-infer-chs-nn | ✅ SUCCESS | >>>>> | |
| UT_Test_ophost | ✅ SUCCESS | ||
| UT_Test_opapi | ✅ SUCCESS | ||
| UT_Test_kernel | ✅ SUCCESS | ||
| UT_Test_opgraph | ✅ SUCCESS | ||
| PreSmoke_A900 | ✅ SUCCESS | >>>>> | |
| PreSmoke_ATK_Test_A2 | ✅ SUCCESS | >>>>> |
[2026-07-07 10:52:07] CI执行结束


流水线任务触发成功
任务链接 [bb2ab487985144629521503f8e17056f][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| codecheck | ✅ SUCCESS | >>>>> | |
| antipoison | ✅ SUCCESS | >>>>> | |
| codecheck_checkpr | ✅ SUCCESS | ||
| StaticCheck_codespell_check | ✅ SUCCESS | ||
| StaticCheck_link_validity_check | ✅ SUCCESS | ||
| StaticCheck_resource_existence_check | ✅ SUCCESS | ||
| StaticCheck_tag_closed_check | ✅ SUCCESS | ||
| StaticCheck_markdownlint | ✅ SUCCESS | ||
| codecheck_style | ✅ SUCCESS | >>>>> | |
| codecheck_precommit | ⚠️ WARNING | >>>>> | >>>>> |
| SCA | ✅ SUCCESS | >>>>> |
[2026-07-07 10:47:28] CI执行结束


/approve


/check


/lgtm


描述
aclnnConvolutionBackward异常场景用例报错描述修改
关联的Issue
#3902
测试
本地验证,冒烟测试,蓝区冒烟,CI pipeline
文档更新
不涉及
类型标签
AI/Agent生成声明