已合并
fix: 规范 random 目录算子日志输出(级别错配/拼写/语法/格式符/中文残留) #5347
fix: 规范 random 目录算子日志输出(级别错配/拼写/语法/格式符/中文残留) #5347
已合并
liangtongxue创建于 15 天前
liangtongxue成员
15 天前

描述

random/ 全目录 26 个算子(含 examples、tests)的日志输出做规范整改,修复以下缺陷:

  • 级别错配(36 处):12 个 geir examples 失败分支 - INFO - [XIR]: xxx failed 改为 - ERROR -,按级别过滤时故障信息不再被忽略
  • 拼写错误:samller→smaller、excepted→expected(语义相反)、RadixSortTIling→RadixSortTiling、Infershape→InferShape 等修正
  • 语法错误(100+ 处):only support→only supports、not support→does not support、should same with→should be the same as、must >→must be greater than、is begin→begins、running begin→started、序数 1th 改写、字符串拼接双空格/逗号粘连等
  • 错误描述不完整(40+ 处):dtype/shape/size/coreNum/ubSize 校验失败补当前实际值;allowed set 补允许集合内容;onnx dtype 支持列表文案与 kvlist 实现对齐
  • 格式符与实参类型不匹配:按头文件实类型逐项核对修正(%lld→%llu、%d→%ld、%u→%lu、%f→%ld 等),消除 -Werror 编译失败与输出垃圾值
  • 魔鬼数字:GE version 裸数字 90000000 统一为 9.0.0(6 处)
  • 中文输出:ST executor/golden.py 中文文案改英文(10 处);Format 支持列表中文顿号改半角逗号
  • 统一日志接口:random_infershape_base.cpp 业务代码 std::cerr 改 OP_LOGD/OP_LOGW/OP_LOGE(4 处);删除 random_dtype_fmt_gen.h 中无调用点的调试打印函数 Print/PrintByColsCore
  • UT 修复:truncated_normal_v2 tiling 用例期望串补齐尾部 3 个字段(RandomUnifiedSimtTilingDataStruct 实为 51 个 int64 字段,期望串原为 48 个;master 上存量失败)

改动方法

仅修改日志语句的文案/级别/格式符与配套标识符拼写(如 isContinous_→isContinuous_ 等全量同步改名),不改动任何校验逻辑与控制流。

关联的Issue

#3083

测试

  • pre-commit 全部通过(clang-format / ruff-check / ruff-format / codespell / OAT 等 10 项)
  • bash build.sh --pkg --jit --soc=ascend950 编译打包通过(含 -Werror=format 严格校验)
  • random 目录全部改动算子 UT:153 个测试全部 PASSED(含修复后的 truncated_normal_v2)
  • 输出文案中文残留、业务代码私有打印残留均为 0

文档更新

类型标签

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

变更摘要

本 PR 依据 cann-log-evaluation 技能对 random/* 全目录(含 examples、tests)执行三轮扫描与整改,核心目的是规范日志与错误文案。改动集中在四类:将 geir 示例中失败分支的日志级别由 INFO 修正为 ERROR;统一并修正日志文案的语法与拼写(如 only supportonly supportsshould same withshould be the same asInfershapeInferShaperunning beginstarted 等);为 dtype/shape/size 校验失败日志补充实际值、为 allowed set 补充集合内容,使描述与实现及校验边界对齐;同时统一日志接口(std::cerrOP_LOGx)、清除中文文案、修正格式符与实际参数类型(如 %ld/%d/%fint64_t/uint32_t/uint64_t)的匹配。

主要改动

  • geir 示例失败分支日志级别整改:在 drop_out_do_mask_v3drop_out_v3stateless_randomrandom_standard_normal_v2random_uniform_int_v2stateless_uniformtruncated_normal_v2 等 12 个 test_geir_*.cpp 示例中,将 Initialize ge/Run graph/Finalize 等失败分支的 printf("%s - INFO - [XIR]: xxx failed") 统一改为 - ERROR -,使失败场景与日志级别匹配。
  • 错误文案语法/拼写规范化drop_out_v3_split_fusion_pass.cppdrop_out_do_mask_tiling_arch35.cppstateless_randperm_tiling_for_sort.cpp 等文件将 only supportonly supportsmust be not equalmust not be equaldtype must be same asmust be the same asInfershapeInferShapeRadixSortTIlingRadixSortTilingrunning beginstarted 等,并修正 get dataType size failFailed to get dataType size. 等表述。
  • 校验失败日志补充当前值drop_out_v3_fusion_pass.cpprandom_tiling_arch35.cppstateless_randperm_tiling_arch35.cppaclnn_multinomial.cppaclnn_bernoulli.cpp 等为 dtype/shape/size/范围校验补充实际取值(如 dtype %d not supportedub size %ld is invalidNumsamples must be greater than 0, but got %ldprob should be in range [0, 1], but got %f),并为 CheckTensor 的 dtype/dimNum 校验失败日志拼接 allowed set 内容。
  • 格式符与实参类型对齐random_tiling_arch35.cpprandom_standard_normal_v2_tiling_arch35.cpprandom_uniform_int_v2_tiling_arch35.cppsim_thread_exponential_tiling_arch22.cppstateless_random_normal_v2_tiling_arch35.cpp 等按成员实类型将 %d/%lld 修正为 %ld/%llu/%u,如 totalCoreNumubSizestateoutputSize 等字段的打印。
  • 日志接口统一与中文清理random_infershape_base.cpp 将业务 std::cerr 改为 OP_LOGx 宏、random_dtype_fmt_gen.h 改为 OP_LOGW;多个 ST executor 与 golden.py 将中文提示(如"非连续操作"、"连接错误")改为英文;各 fusion pass 将 GE 版本魔鬼数字 90000000 统一表述为 9.0.0,并修正 random_uniform_onnx_plugin.cpp 中 ONNX dtype 注释/文案与 kvlist 实现不一致的问题。
likedislike
不准确?
atomgit-bot
atomgit-bot
15 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
15 天前 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
15 天前 添加了label:cann-cla/yes
此处折叠了101条消息 查看更多
CANN-robotCANN-robot成员
11 天前 添加了label:approved
songkai111成员
11 天前 评论:

/lgtm

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