Closed
[Bug-Report|缺陷反馈]: CircularPad op_api UT 4 处返回码断言被注释,需逐例核验恢复 #4291
陈思Add on  11 days agoClosed at  4 days ago
陈思
陈思Collaborators
11 days ago Create

一、问题描述

从原 Issue #4199 按算子责任人拆分,原始反馈者为 @wsnidie1。

固定版本 8c1f633e3679f83b55a237c552f842cdcf80462f 中,下列存活测试体仍调用 TestGetWorkspaceSize,但相应 EXPECT_EQ(aclRet, ...) 被整行注释,存在显式返回码断言覆盖缺口。注释原因及原预期正确性仍待核实。

文件 原报告行号
conversion/circular_pad/tests/ut/op_api/test_aclnn_circular_pad2d.cpp 36、153
conversion/circular_pad/tests/ut/op_api/test_aclnn_circular_pad3d.cpp 36、167

共 4 处。二维、三维各涉及普通输入和 dtype 循环的返回码断言;一处循环覆盖多个 dtype,4 是源码位置数。

二、环境信息

源码核查基线如上;实际 UT runner、SOC 和 CANN 环境需在验证记录中明确,尚未运行目标 UT。

三、重现步骤

  1. 在所列源码位置确认被注释断言及所在存活用例。
  2. 记录各例的 SOC、shape、dtype、format,核对接口契约与原期望。
  3. 逐例恢复正确断言并运行对应 UT/CI,记录预期与实际返回值。

四、预期结果

恢复有效返回码检查并提供验证结果;原预期过时时区分测试与实现应修改的一侧。受已知缺陷或平台条件阻塞时,明确原因、平台和关联任务,必要时按最小用例范围显式跳过,避免保留无说明的注释断言。成功与错误码用例分别核验,不统一改成 ACL_SUCCESS。

本单跟踪返回码覆盖恢复;精度验证另行评估。处理结果请回链 #4199。

likedislike
陈思陈思Collaborators
11 days ago Added label: bug-report
陈思陈思Collaborators
11 days ago Assign yue-ma as responsible person
陈思陈思Collaborators
11 days ago Added label: pending-confirmation
陈思陈思Collaborators
11 days ago Removed label: pending-confirmation
陈思
陈思Collaborators
10 days ago Comments:

您好,已核对 cann/ops-math 提交 212ce9d5460b1be26cfbee481aff1acbf5cc5c62:报告的4处成功返回码断言仍被注释,且两份测试文件与所列 8c1f633e… 基线一致。具体是 2d 的 case_1、case_8,以及 3d 的 case_1、case_9;其中两处循环分别覆盖5种 dtype。

这些用例的 ND 格式、输入维数、padding 长度及输出尺寸,与当前接口检查静态吻合,3d 循环使用4维输入也在允许范围内。一个可行方向是逐例恢复正确的成功断言,并在明确的 SOC/CANN/UT runner 下核对实际返回;若出现失败,再区分环境、实现路径与测试预期的问题,保留现有非法输入的错误码检查。

当前确认的是显式断言覆盖缺口,尚未运行这些 UT。后续验证结论可在本项记录并回链 #4199。

likedislike
WangJinCheng
WangJinChengCollaborators
5 days ago Comments:

/assign

likedislike
CANN-robotCANN-robotCollaborators
5 days ago Assign Anthony0331 as responsible person
Yyue-maCollaborators
5 days ago Removed responsible person yue-ma
WangJinChengWangJinChengCollaborators
4 days ago Associated pull request
CANN-robotCANN-robotCollaborators
4 days ago Close issue
CANN-robotCANN-robotCollaborators
4 days ago Added label: resolved