已合并
【PR】: use Internal error instead inner error #3682
zhanj创建于 7月16日
【PR】: use Internal error instead inner error #3682
已合并
zhanj创建于 7月16日
zhanj
zhanj成员
7月16日

Pull Request

描述

https://gitcode.com/cann/runtime/pull/3682

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhanj 的贡献)
zhanjzhanj成员
7月16日 添加了label:enhancement
zhanjzhanj成员
7月16日 创建了 pull request,commit 357424e3
atomgit-bot
atomgit-bot
7月16日 评论:

变更摘要

本次 PR 是一个小型的文案修正,将 error_manager.cc 中两处错误提示信息从 "Inner Error!" 统一改为 "Internal error!",使错误消息的措辞更加规范和一致。

主要改动

  • AssembleInnerErrorMessage 函数中的错误提示修正:将拼接错误信息时追加的 ": Inner Error!" 改为 ": Internal error!",统一错误消息措辞。
  • OutputErrMessage 函数中的默认错误提示修正:将空错误消息场景下输出的 "E19999: Inner Error!" 改为 "E19999: Internal error!",保持与上述修改一致的表述风格。
likedislike
不准确?
atomgit-bot
atomgit-bot
7月16日 评论:

代码审查

审查总结

本次审查覆盖了 src/dfx/error_manager/error_manager.cc 中的全部变更(2 行修改),共发现 1 个问题。

优先级 数量
P0 0
P1 0
P2 1
P3 0

变更文件确认

  • src/dfx/error_manager/error_manager.cc:发现 1 个问题(字符串变更不完整,平行 C 实现和文档未同步更新)

整体风险评估:此 diff 变更本身逻辑正确,仅将用户可见的错误消息从 "Inner Error!" 改为 "Internal error!"。主要风险在于变更范围不完整 — 项目中平行 C 实现(runtime_compact/c_base/src/error_manager.c)和中文文档仍保留旧字符串,可能导致不同运行时输出不一致,给用户和下游工具带来困惑。建议补充更新 C 实现和文档以完成全局替换。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
不准确?
CANN-robotCANN-robot成员
7月16日 添加了label:cann-cla/yes
zhanj
zhanj成员
7月16日 评论:

compile

likedislike
CANN-robot
CANN-robot成员
7月16日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
7月16日 评论:

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


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
src/dfx zhuliangying, 卢煜坤 (2/2) zhuliangying (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
zhanjzhanj成员
7月16日 update merge request[project id: 8810053, iid: 3682, commit_id: aac689c8c534a52ec067d165e3c2994cebcced6e] virtual merging success
CANN-robotCANN-robot成员
7月16日 将newstarzj,chenhao_1209,zhuliangying,wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,ykl999,houyanbao,tingwood设为评审人
CANN-robotCANN-robot成员
7月16日 将newstarzj,chenhao_1209,zhuliangying设为审查人
CANN-robotCANN-robot成员
7月16日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
7月16日 评论:

流水线任务触发成功
任务链接 [e4464a0048b54483bd3189cefc442366][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_msprof_part1 ✅ SUCCESS
UT_Test_msprof_part2 ✅ SUCCESS
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
UT_Test_camodel_check ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS >>>>>

[2026-07-16 16:08:06]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月16日 评论:

流水线任务触发成功
任务链接 [5c902324cb4e4f4283fedd2b8ad740a2][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-07-16 15:57:28]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:api-check-pass
CANN-robotCANN-robot成员
7月16日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7月16日 添加了label:ci-pipeline-passed
zhanjzhanj成员
7月16日 修改了pull request 的描述
zhuliangying成员
7月16日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:approved
ykl999
ykl999成员
7月16日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:lgtm
CANN-robotCANN-robot成员
7月16日 合入了pull request