已合并
fix: 整改责任田算子日志质量(修复日志规范问题) #5023
fix: 整改责任田算子日志质量(修复日志规范问题) #5023
已合并
StoneChan_创建于 9 天前
StoneChan_
StoneChan_成员
9 天前

一、问题描述 (必填)

依据 CANN 日志规范检查规则,对责任田算子的运行时日志进行质量整改,修复以下问题:

  • 级别错配:GE IR 示例(test_geir_*.cpp)失败信息误标 INFO(应 ERROR),存在故障被忽略风险
  • 拼写错误:caculate/negtive/faild/mat 等
  • 语法错误:"is failed" 中式结构、"fail to get" 缺时态、缺助动词/介词、must same 等
  • 变量越限日志缺正确范围(未给出期望值)
  • 中文标点(全角顿号)残留
  • 示例直接 printf/std::cout 绕过统一日志接口;内部缩写 [XIR] 无法理解
  • UT 测试中冗余 std::cout 状态输出

影响算子:cumsum/view_copy/addr/split/split_v/unpack/adds/concat/pack/as_strided/eye/dynamic_partition/assign/masked_fill/strided_slice_grad/triu/tril/tensor_move/asin/power/concat_d/concat_v2

二、环境信息 (可选)

不涉及(纯日志文案整改,无运行时行为变化)

三、重现步骤 (可选)

执行算子 UT / 查看 logs/plog 中相关算子日志

四、预期结果 (可选)

日志级别、文案符合日志规范,便于问题定位

💡 备注(选填):排除 experimental 目录,仅整改主仓

关联 PR:https://gitcode.com/cann/ops-math/merge_requests/5023


关联的Issue:#2845

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 StoneChan_ 的贡献)
StoneChan_StoneChan_成员
9 天前 创建了 pull request,commit 736f0768
atomgit-bot
atomgit-bot
9 天前 评论:

变更摘要

本 PR 依据 CANN 日志规范检查规则,对 cumsum/view_copy/addr/split/split_v/unpack/adds/concat/pack/as_strided/eye/dynamic_partition/assign/masked_fill/strided_slice_grad/triu/tril/tensor_move/asin/power/concat_d/concat_v2 等责任田算子进行运行时日志质量整改(排除 experimental 目录)。核心内容为:将 GE IR 示例(test_geir_*.cpp)中失败信息误标 INFO 的级别统一改为 ERROR;将直接 printf/std::cout 统一收敛到 LOG_PRINT 宏(缺失处新增),并将内部缩写 [XIR] 替换为算子专属标记(如 [AS_STRIDED_GE_IR][SPLIT_GE_IR][CUMSUM_GE_IR] 等);同时清理 UT 测试中冗余的 std::cout 状态输出。本次整改均为日志文案/级别调整,不改变算子功能逻辑。

主要改动

  • GE IR 示例日志级别与输出方式规范化: 在 test_geir_as_strided.cpptest_geir_concat.cpptest_geir_split.cpptest_geir_cumsum.cpp 等示例中,将 GE 初始化失败、RunGraph 失败、GEFinalize 失败等日志由 INFO 改为 ERROR,并将 printf/std::cout 统一替换为新增的 LOG_PRINT 宏(如 LOG_PRINT("input %d shape size: %ld\n", i, input_shape)),日志标记由 [XIR] 改为算子专属标记(如 [CONCAT_GE_IR][ASIN_GE_IR])。
  • UT 测试冗余输出清理: 删除各算子 UT(如 test_as_strided_tiling.cpptest_assign_tiling_arch35.cpptest_aclnn_tril.cpptest_triu.cpp 等)中 SetUpTestCase/TearDownTestCase 及循环用例内的 std::cout 状态输出;test_split_v_tiling.cpp 中为 nBlockCountmBlockCount 补充 (void) 强转以避免未使用告警。
  • 日志文案语法/拼写规范化: 将 "is failed" 中式结构、"fail to get"/"fail." 缺时态等统一改为 "Failed to ..." 标准表达,例如 dynamic_partition_tiling.cpp 的 "Failed to set tiling data!"、split_tiling_arch35.cpp 的 "Failed to get ubSize"、assign_tiling_arch35.cpp 的 "Failed to set tiling data.",并修正 "faild"→"failed"、"negtive"→"negative"、"socversion"→"soc version" 等拼写。
  • 越限校验日志补充正确范围: split_v/unpackubSize 校验日志补充 "ubSize must be greater than 0.",eye_infershape.cppnum_rows/batch_shape 校验日志补充 "must be greater than 0, but got %ld",使越限提示包含具体取值范围。
  • 中文标点与格式残留修复: aclnn_cat.cpp 的 "Format only support ND、NCHW、..." 改为 "Format only supports ND, NCHW, ..."(全角顿号改半角逗号),concat_tiling_arch35.cppview_copy_tiling_arch35.cpp 的 tiling 打印日志中逗号/空格格式规范化,并修正 "input_size must ge 1" 为 "must be at least 1"、split_infershape.cpp "must be divided by" 为 "must be divisible by"。
likedislike
atomgit-bot
atomgit-bot
9 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
9 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9 天前 评论:

CLA Signature Pass

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

likedislike
此处折叠了73条消息 查看更多
CANN-robotCANN-robot成员
8 天前 添加了label:approved
songkai111成员
8 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:lgtm
CANN-robotCANN-robot成员
8 天前 关闭了关联的issue
CANN-robotCANN-robot成员
8 天前 合入了pull request