已合并
【PR】: [fix] [autofuse] fix codecheck. #4787
邢智雄创建于 7 小时前
【PR】: [fix] [autofuse] fix codecheck. #4787
已合并
邢智雄创建于 7 小时前
邢智雄
邢智雄成员
7 小时前

Pull Request

描述

删除注释代码 并 校验输入文件的路径。

变更类型

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

关联的Issue

如何测试

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

  1. codecheck告警修复,ci 流水线和rdv pass即可

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 邢智雄 的贡献)
邢智雄邢智雄成员
7 小时前 添加了label:enhancement
邢智雄邢智雄成员
7 小时前 创建了 pull request,commit 95abb08b
atomgit-bot
atomgit-bot
7 小时前 评论:

变更摘要

PR 的标题与描述均为模板占位内容,未提供具体业务背景;结合代码改动可知,本次变更主要涉及 autofuse 模块下配置解析与后处理两处调整:一是在 ParseSkipNodeNamesConfig 中先通过 RealPath 解析配置文件真实路径并做空值校验,再打开配置文件;二是清理 CubeFixpip 中被注释掉的删除节点代码残留。整体为路径解析加固与代码清理性质的小改动,不改动核心业务流程。

主要改动

  • 配置文件真实路径解析: ParseSkipNodeNamesConfig 在打开配置文件前新增 RealPath(config_path.c_str()) 调用,解析失败(返回空串)时记录 GELOGW 警告并直接返回,解析成功后才以 real_config_path 构造 std::ifstream 打开文件,增强配置路径解析的健壮性。
  • 新增头文件依赖: auto_fuse_config.h 增加 graph/utils/file_utils.h 头文件引用,为调用 RealPath 接口提供依赖声明。
  • 清理注释残留代码: CubeFixpip 中删除了已注释掉的 asc_adapt::DelNode(graph, peer_in_node) 行及相应说明注释,属于纯注释清理,不影响现有逻辑(当前 relu 节点保留以兼容不支持 fixpip 的 tiling key)。
likedislike
不准确?
atomgit-bot
atomgit-bot
7 小时前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
7 小时前 添加了label:cann-cla/yes
邢智雄
邢智雄成员
7 小时前 评论:

compile

likedislike
邢智雄邢智雄成员
7 小时前 update merge request[project id: 8824148, iid: 4787, commit_id: 1c1bb18b2a78ff47821a30e4c9cd7caa3a27a801] virtual merging success
CANN-robot
CANN-robot成员
7 小时前 评论:

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, xchu42, ClarkXie (3/2) yangyongqiang0606 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7 小时前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
7 小时前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
CANN-robot
CANN-robot成员
7 小时前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
7 小时前 添加了label:ci-pipeline-running
xchu42
xchu42成员
7 小时前 评论:

/lgtm

likedislike
邢智雄邢智雄成员
7 小时前 修改了pull request 的描述
此处折叠了9条事件消息 查看更多
邢智雄邢智雄成员
7 小时前 修改标题为 “【PR】: [fix] [autofuse] fix codecheck.”,原标题为“【PR】: [autofuse] fix codecheck.”
ClarkXie成员
7 小时前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7 小时前 添加了label:lgtm
邢智雄
邢智雄成员
7 小时前 评论:

/check-pr

likedislike
CANN-robot
CANN-robot成员
7 小时前 评论:

The following label is not ready.

approved: Please wait for committers to review the code.

likedislike
yangyongqiang
yangyongqiang成员
6 小时前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
6 小时前 添加了label:approved
CANN-robotCANN-robot成员
6 小时前 合入了pull request