已合并
【PR】: 整改目录日志不规范问题 #5025
lihecen创建于 20 天前
【PR】: 整改目录日志不规范问题 #5025
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 lihecen 的贡献）20 天前 关联了issue：[Log规范] runtime目录下日志不规范问题整改
atomgit-bot
20 天前 评论：
20 天前 评论：
变更摘要
本 PR 按照 issue #566 的日志规范扫描结果，对 api、compiler、runtime 目录下的日志文案、级别与格式进行整改，涉及拼写、语法、字符串拼接、日志级别和度量单位等几类问题，不涉及功能逻辑改动。核心变更包括：将 om2 format_transfers 中的 Dose not support 修正为 Does not support，将 uninfered/infered 修正为 uninferred/inferred，将 Not find、Failed to lowering、not support dtype、dim num not match 等表述修正为规范英文；对部分失败场景日志级别由 GELOGI 调整为 GELOGW；并补充超时日志的秒单位。
主要改动
- 拼写与语法修正:
runtime/om2/formats/format_transfers下多处CheckArgs的错误信息由Dose not support trans format改为Does not support trans format；api/session/jit_execution/utils/jit_infer_utils.cc与binary_partitioner.cc中uninfered/infered改为uninferred/inferred，并同步更新注释。 - 错误信息表述规范化:
aicore_node_converter.cc中Not find AI core task def改为Failed to find AI core task def；if_case_converter.cc中Failed to lowering改为Failed to lower；build_tensor.cc中not support dtype改为dtype is not supported；infer_shape_range.cc中dim num not match改为dim num does not match；binary_partitioner.cc中remaining nodes is empty改为remaining nodes are empty。 - 相邻字符串拼接补空格:
graph_manager.cc的OPTION_EXEC_HCOM_GROUPLIST_V2冲突提示、lowerings.cpp的 re-lowering 提示中补齐拼接处缺失的空格，避免日志文案粘连。 - 日志级别错配修正:
python_custom_op_bridge_loader.cc中opendir/stat失败日志、om2_model_executor.cc中资源释放失败日志、rt_v2_pipeline_executor.cc中stage停止失败分支由GELOGI改为GELOGW（后者还拆分为成功/失败两条分支日志）。 - 补充度量单位:
model_manager.cc中LaunchKernelCustAicpuSo与LaunchKernelBuiltinAicpuSo的timeout日志格式由%u改为%us，为超时值补充秒单位。
不准确？
atomgit-bot
20 天前 评论：
20 天前 评论：
20 天前 添加了label：stat/needs-squash
20 天前 添加了label：cann-cla/yes
CANN-robot
20 天前 评论：
20 天前 评论：
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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-cann/ge | ✅ yangyongqiang0606, ClarkXie, Mexyy (3/2) | ✅ yangyongqiang0606 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
lihecen, thanks for your pull request. All authors of the commits have signed the CLA. 👍
CANN-robot20 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
20 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robot20 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
20 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
lihecen
20 天前 评论：
20 天前 评论：
compile
20 天前 预合并成功（commit_id: d7aa40036c623a158e87cd76b02a2d9512ed25f5）
CANN-robot
20 天前 评论：
20 天前 评论：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
20 天前 添加了label：ci-pipeline-running
20 天前 删除了label：ci-pipeline-running
20 天前 添加了label：ci-pipeline-passed
20 天前 预合并成功（commit_id: 464706f3d18f081a94fec7a5a276d31c4370d103）
20 天前 预合并成功（commit_id: 26d70847da7750a01d4daebf662ee253f0623972）
lihecen
20 天前 评论：
20 天前 评论：
compile
20 天前 预合并成功（commit_id: bba0cfd46ba7475cd25d3ffabed277053cdca970）
20 天前 删除了label：cann-cla/yes
20 天前 删除了label：ci-pipeline-passedstat/needs-squash
CANN-robot
20 天前 评论：
20 天前 评论：
20 天前 添加了label：cann-cla/yes
CANN-robot
20 天前 评论：
20 天前 评论：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
20 天前 添加了label：ci-pipeline-running
此处折叠了5条事件消息 查看更多
20 天前 添加了label：ci-pipeline-passed
ClarkXie
20 天前 评论：
20 天前 评论：
/lgtm
20 天前 添加了label：lgtm
yangyongqiang
20 天前 评论：
20 天前 评论：
/approve
20 天前 添加了label：approved
20 天前 合入了pull request
Pull Request
描述
按照 issue #566 的日志规范扫描结果，整改 api、compiler、runtime 目录下共 25 处日志不规范问题：
Dose not support→Does not support（runtime/om2 format_transfers 8处）、uninfered→uninferred（api/session 2处）Not find→Failed to find（aicore_node_converter 4处）、Failed to lowering→Failed to lower（if_case_converter 5处）、not support dtype→dtype is not supported（build_tensor）、dim num not match→dim num does not match（infer_shape_range）、nodes is empty→nodes are empty（binary_partitioner）、is not support→is not supported（transpose 2处，此前已修复）s（秒）变更类型
关联的Issue
#566
如何测试
核对清单
其他信息
issue #566 中
base/asc/tilingdata_base_impl.cc属于 metadef 独立仓库，不在本仓库修复范围内。