Star707
465
代码
代码
Issues36
Pull Requests111
Actions
项目讨论
Wiki
分析
Star707
465
已合并
【fix】: remove aclgrphCalibration and set rt2 runinternal thread name #4038
tang-haojie创建于 7月16日
【fix】: remove aclgrphCalibration and set rt2 runinternal thread name #4038
已合并
tang-haojie创建于 7月16日
从tang-haojie/gedevelop合入到cann/gedevelop
T
tang-haojie成员
7月16日

Pull Request

描述

aclgrphCalibration接口不在ge仓，相关docs和readme也删除。
HybridModelAsyncExecutor::RunInternal这个线程函数没有设置线程名，此次加上

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 tang-haojie 的贡献）
Ttang-haojie成员
7月16日 添加了label：enhancement
Ttang-haojie成员
7月16日 创建了 pull request，commit 67280ce5
atomgit-bot
atomgit-bot
7月16日 评论：

变更摘要

此 PR 主要包含两项变更：移除 aclgrphCalibration 相关的校准代码（净删除约 131 行），以及为 rt2 运行时内部线程设置可辨识的线程名称，便于调试和问题定位。

主要改动

  • 移除 aclgrphCalibration 校准模块：从 3 个文件中删除了约 131 行代码，涉及 aclgrphCalibration 相关的校准逻辑，简化了运行时代码。
  • 设置 rt2 运行时线程名称：在 HybridModelAsyncExecutor::RunInternal() 中添加了 SET_THREAD_NAME(pthread_self(), "ge_rt2_mdlrun") 调用，将 rt2 模型运行的内部线程命名为 ge_rt2_mdlrun，便于在调试器中识别该线程。
likedislike
不准确？
atomgit-bot
atomgit-bot
7月16日 评论：

代码审查

审查总结

本次 PR 共涉及 4 个文件变更（含 1 个删除），审查结果如下：

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

各文件审查结论：

  • runtime/v1/hybrid/executor/hybrid_model_async_executor.cc：发现 1 个 P0 问题 — 新增 SET_THREAD_NAME 调用但缺少必要的 #include "common/compile_profiling/ge_call_wrapper.h"，将导致编译失败。
  • docs/zh/api/graph_engine_api/README.md：仅从目录中移除对已删除文件的引用，无问题。
  • docs/zh/api/graph_engine_api/cpp/ge/aclgrphCalibration.md：整个文件被删除，属于 API 移除的配套清理，无问题。
  • docs/zh/user_guides/graph_dev/more_features/quantization.md：移除"自动量化"章节及其对 aclgrphCalibration 的引用，全文无残留引用，无问题。

整体风险评估： 该 PR 的主要风险在于 C++ 编译失败（P0），需在合入前补充缺失的头文件引用。文档变更部分干净无副作用。

类型 数量
🔴 阻塞 1
🟡 建议 0

⛔ 需要修改

likedislike
不准确？
CANN-robotCANN-robot成员
7月16日 添加了label：cann-cla/yes
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
repo-cann/ge ✅ zhanj, Mexyy, yuht9 (3/2) ✅ zhanj (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7月16日 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
此处折叠了9条事件消息 查看更多
Ttang-haojie成员
7月16日 修改了pull request 的描述
T
tang-haojie成员
7月16日 评论：

compile

likedislike
Ttang-haojie成员
7月16日 update merge request[project id: 8824148, iid: 4038, commit_id: 4d23824a1e820c72da2bf93df1848bf1a3cc768d] virtual merging success
Ttang-haojie成员
7月16日 修改标题为 “【fix】: remove aclgrphCalibration and set rt2 runinternal thread name”，原标题为“【PR】: remove aclgrphCalibration and set rt2 runinternal thread name”
CANN-robotCANN-robot成员
7月16日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月16日 评论：

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ❌ FAILED >>>>>
Compile_ARM_executor ❌ FAILED >>>>>
Compile_ARM_dflow ❌ FAILED >>>>>
likedislike
CANN-robot
CANN-robot成员
7月16日 评论：

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ❌ FAILED >>>>>
Compile_X86_executor ❌ FAILED >>>>>
Compile_X86_dflow ❌ FAILED >>>>>
likedislike
CANN-robot
CANN-robot成员
7月16日 评论：

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
codecheck ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ 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 17:06:59]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月16日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
7月16日 添加了label：ci-pipeline-failed
Ttang-haojie成员
7月16日 update merge request[project id: 8824148, iid: 4038, commit_id: 6ed036659f4ecb4e7b80963ad0f536a7dc6eb4e5] virtual merging success
Ttang-haojie成员
7月16日 强制推送  1 个提交:54df9a95-remove aclgrphCalibration and set rt2 runinternal thread name
Ttang-haojie成员
7月16日 update merge request[project id: 8824148, iid: 4038, commit_id: 54ee0cc71e39cf5201f322dd7250f7db9cdbe777] virtual merging success
T
tang-haojie成员
7月16日 评论：

compile

likedislike
Ttang-haojie成员
7月16日 update merge request[project id: 8824148, iid: 4038, commit_id: 66ea6d113cae1b257a259b58a7567314fe857d3d] virtual merging success
CANN-robotCANN-robot成员
7月16日 删除了label：ci-pipeline-failed
CANN-robotCANN-robot成员
7月16日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月16日 评论：

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-07-16 17:15:19]    CI执行结束

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

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-07-16 17:21:40]    CI执行结束

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

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
codecheck ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ 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 17:13:25]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label：api-check-pass
atomgit-bot
atomgit-bot7月16日进行代码检视1
runtime/v1/hybrid/executor/hybrid_model_async_executor.cc
@@ -9,6 +9,7 @@
99 */
1010 
1111#include "hybrid/executor/hybrid_model_async_executor.h"
12+#include "common/compile_profiling/ge_call_wrapper.h"
1213#include "base/err_mgr.h"
1314#include "graph/load/model_manager/model_utils.h"
1415#include "graph/utils/tensor_utils.h"
@@ -230,6 +231,7 @@ Status HybridModelAsyncExecutor::Init(const aclrtStream stream) {
230231}
231232 
232233Status HybridModelAsyncExecutor::RunInternal() {
atomgit-bot
atomgit-bot7月16日评论：

🔴 Critical

在 RunInternal() 方法中新增了对 SET_THREAD_NAME(pthread_self(), "ge_rt2_mdlrun") 的调用（第 233 行），但该文件未引入定义该宏的头文件 common/compile_profiling/ge_call_wrapper.h。

通过检查项目内其他使用 SET_THREAD_NAME 的文件（model_executor.cc、davinci_model.cc、rt_v2_stage_state.cc、hybrid_model_rt_v2_executor.cc 等），它们均显式 #include "common/compile_profiling/ge_call_wrapper.h"。当前文件仅引入了 common/profiling_definitions.h，该头文件不包含 ge_call_wrapper.h，也未通过其他包含链间接引入。

后果：编译时 SET_THREAD_NAME 为未定义标识符，导致编译失败。

建议：在文件头部 #include 区域添加对 ge_call_wrapper.h 的引用，参照其他使用 SET_THREAD_NAME 的文件的标准做法。建议加在 #include "common/profiling_definitions.h" 之后。

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
Y
yuht9成员
7月16日 评论：

/lgtm

likedislike
夏
夏国正成员
7月16日 评论：

/lgtm

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label：lgtm
zhanj
zhanj成员
7月16日 评论：

/approve

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label：approved
CANN-robotCANN-robot成员
7月16日 解决了最后一个问题
CANN-robotCANN-robot成员
7月16日 合入了pull request