已合并
fix:improve log usability-0907 #4771
fix:improve log usability-0907 #4771
已合并
洪跃城创建于 7 天前
洪跃城
洪跃城成员
7 天前

Pull Request

描述

本次提交基于 CANN 日志易用性评测结果,对 GE 仓库 develop 分支中的日志文案进行整改。

本次整改排除了提示级问题,以及 om2/engines/tests/api/atc/graph_eager 等禁改目录和已确认不需要修改的工具、生成代码文件。最终共修改 15 个文件,覆盖 Excel 中 27 条问题,不涉及业务逻辑、控制流或算法变更。

1. 致命 - 级别错配(2 处)

  • examples/fusion_pass/.../tf_forward.py:36:78
  • INFO 级日志中含跨级别词 error,改为中性描述 diff

2. 严重 - 拼写错误(22处,10个文件)

修正前 修正后
shoulde should
subsequenet subsequent
indice indices
parmas params
scheding scheduling
rang range
tenser tensor
transfer-ing transfer

3. 一般 - 禁止自定义打印 / 缺少度量单位 / 语法错误

  • print() / std::cout / printf 改为项目统一日志接口(GELOGI/GELOGE/logging),保留原文案语义
  • 为内存/容量数值补充单位(如 size:500size:500 MB
  • 修正重复单词(value valuevalue

变更类型

请选择本次引入的变更类型:

关联的Issue

无。

如何测试

描述测试此变更的步骤和前提条件:

  1. 运行项目 pre-commit 检查,确认格式、拼写和合规检查通过。
  2. 运行 PR 编译及 UT/ST 流水线,确认相关门禁通过。
  3. 检查代码差异,确认仅修改日志文案,不涉及业务逻辑、控制流和算法。

核对清单

其他信息

本次整改仅修改日志文案与日志输出接口,不改变任何业务逻辑、控制流、算法。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 洪跃城 的贡献)
洪跃城洪跃城成员
7 天前 添加了label:enhancement
洪跃城洪跃城成员
7 天前 创建了 pull request,commit 5a1b42f2
atomgit-bot
atomgit-bot
7 天前 评论:

🤖 正在生成合并请求摘要,请稍候…

likedislike
不准确?
atomgit-bot
atomgit-bot
7 天前 评论:

🤖 AI 代码检视正在进行中,请稍候…

likedislike
不准确?
CANN-robotCANN-robot成员
7 天前 添加了label:stat/needs-squash
此处折叠了116条消息 查看更多
洪跃城洪跃城成员
6 天前 解决了最后一个问题
洪跃城洪跃城成员
6 天前 解决了最后一个问题
yangyongqiang
yangyongqiang成员
5 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:approved
CANN-robotCANN-robot成员
5 天前 合入了pull request