已合并
【PR】: 将parser下的model_saver.h和pass_manager.h重命名 #4447
chenqian1365创建于 4 天前
【PR】: 将parser下的model_saver.h和pass_manager.h重命名 #4447
已合并
chenqian1365创建于 4 天前
chenqian1365成员
4 天前

Pull Request

描述

将parer下的model_saver.h和pass_manager.h重命名

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 chenqian1365 的贡献)
Cchenqian1365成员
4 天前 添加了label:enhancement
Cchenqian1365成员
4 天前 创建了 pull request,commit 391b9f12
atomgit-bot
atomgit-bot
4 天前 评论:

变更摘要

本 PR 主要对 parser 模块下的两个头文件进行重命名,以统一命名规范(将 model_saver.h 重命名为 parser_model_saver.h、将 pass_manager.h 重命名为 parser_pass_manager.h),并同步更新了各解析器实现文件与测试文件中对应的 #include 引用,属于纯命名层面的改动,未涉及业务逻辑变更。

主要改动

  • 头文件重命名:将 parser/common/model_saver.h 重命名为 parser/common/parser_model_saver.h,将 parser/common/pass_manager.h 重命名为 parser/common/parser_pass_manager.h
  • 更新头文件保护宏parser_model_saver.h 的宏由 PARSER_COMMON_FILE_SAVER_H_ 改为 PARSER_COMMON_PARSER_MODEL_SAVER_H_parser_pass_manager.h 的宏由 PARSER_COMMON_PASS_MANAGER_H_ 改为 PARSER_COMMON_PARSER_PASS_MANAGER_H_,避免与旧名冲突。
  • 同步更新源码引用caffe_parser.cconnx_parser.cctensorflow_parser.cctensorflow_custom_op_parser.ccpre_checker.ccmodel_saver.ccpass_manager.cc 等文件的 #include 均改为引用新头文件名。
  • 同步更新测试文件引用test_tensorflow_parser.ccacl_graph_parser_unittest.cctensorflow_parser_unittest.cc 等测试文件中的头文件引用同步改为新文件名,保证测试编译与源码一致。
likedislike
atomgit-bot
atomgit-bot
4 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
4 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
4 天前 评论:

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.
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 张德鹏, tangqunzhang, yangyongqiang0606 (3/2) 张德鹏 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robot
CANN-robot成员
4 天前 评论:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
4 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
4 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
chenqian1365成员
4 天前 评论:

compile

likedislike
Cchenqian1365成员
4 天前 update merge request[project id: 8824148, iid: 4447, commit_id: d7466a21ec4d88f00451601a1b1933a437218739] virtual merging success
CANN-robotCANN-robot成员
4 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
4 天前 评论:

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

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

[2026-08-18 20:50:02]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
4 天前 评论:

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

任务名称状态日志下载链接
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-08-18 20:57:15]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
4 天前 评论:

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

任务名称状态日志下载链接
codecheck_Pr ✅ 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-08-18 20:47:08]    CI执行结束

likedislike
CANN-robotCANN-robot成员
4 天前 添加了label:api-check-pass
此处折叠了7条事件消息 查看更多
Cchenqian1365成员
3 天前 修改标题为 “【PR】: 将parser下的model_saver.h和pass_manager.h重命名”,原标题为“【PR】: 将parer下的model_saver.h和pass_manager.h重命名”
tangqunzhang成员
3 天前 评论:

/lgtm

likedislike
tangqunzhang成员
3 天前 评论:

【MR评价】评价分数:3,
评价意见:
编码规范遵守度:无编码规范问题;
代码设计:不涉及代码设计;
DT质量:基本逻辑DT测试覆盖充足;

likedislike
yangyongqiang
yangyongqiang成员
3 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:lgtm
张德鹏成员
3 天前 评论:

/approve

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