Star634
237
代码
代码
Issues35
Pull Requests88
Actions
项目讨论
Wiki
分析
Star634
237
已合并
【PR】: 修改tf安装脚本路径为scripts/env_install/tensorflow/setup_tf_env.sh #1461
zzq创建于 7月24日
【PR】: 修改tf安装脚本路径为scripts/env_install/tensorflow/setup_tf_env.sh #1461
已合并
zzq创建于 7月24日
从gcw_Xt6hqG9y/graph-autofusion_7464tensorflow_develop合入到cann/graph-autofusiondevelop
Z
zzq成员
7月24日

Pull Request

描述

修改tf安装脚本路径为scripts/env_install/tensorflow/setup_tf_env.sh

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 zzq 的贡献）
Zzzq成员
7月24日 添加了label：enhancement
Zzzq成员
7月24日 创建了 pull request，commit 1c8cf16d
atomgit-bot
atomgit-bot
7月24日 评论：

变更摘要

此 PR 主要将 TensorFlow 环境安装脚本重新组织到新路径 scripts/env_install/tensorflow/setup_tf_env.sh 下，并同步更新了其他文件中对该脚本的路径引用，使项目目录结构更加清晰。

主要改动

  • 脚本文件重命名/迁移：将原有的 TF 安装脚本移动至 scripts/env_install/tensorflow/setup_tf_env.sh，统一归入 env_install 目录下的 tensorflow 子目录。
  • 路径引用更新：其他 2 个文件中对 TF 安装脚本的旧路径引用同步修改为新的 scripts/env_install/tensorflow/setup_tf_env.sh，确保链接一致性。
likedislike
atomgit-bot
atomgit-bot
7月24日 评论：

代码审查

审查总结

本次 PR 将 setup_tf_env.sh 从 scripts/env_install/ 移动到 scripts/env_install/tensorflow/，并更新了两份文档中的脚本调用路径。审查覆盖了全部 3 个变更文件：

文件 审查结果
scripts/env_install/tensorflow/setup_tf_env.sh 纯重命名，内容未变更，无新增问题
autofuse/examples/tensorflow/README.md 脚本路径已更新正确，但激活脚本路径未同步更新（P2）
docs/env_install/tensorflow/env_tf.md 脚本路径已更新正确，但激活脚本路径未同步更新（P2）

发现统计：P2 × 2

整体风险评估：中等。核心问题是：脚本移动后默认安装根目录下沉了一级（scripts/env_install/env/ → scripts/env_install/tensorflow/env/），但两份文档中的 source scripts/env_install/env/activate_tf*.sh 激活命令未同步更新。用户按文档操作会直接遇到 "No such file or directory" 错误。修复仅需将 4 处激活路径同步更新即可。

附注：autofuse/examples/tensorflow/af_tf_eleandele/README.md（第 18、22 行）同样引用了旧激活路径，同样会受此变更影响而失效，但该文件不在本次 PR 的变更范围内，建议一并修复。

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

💬 仅评论

likedislike
CANN-robotCANN-robot成员
7月24日 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
7月24日 评论：

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/graph-autofusion ✅ zhanj, xchu42, xuyafei, zhang_shengjie (4/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

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

likedislike
CANN-robotCANN-robot成员
7月24日 将yangyongqiang0606,xchu42,zhang_shengjie,depeng1994,wqtshg_wt,peiyang,shengnan666,XuebinYang,sjtulxh,zhanj,zhujingjing,xuyafei,wangxiaotian995设为评审人
CANN-robotCANN-robot成员
7月24日 将yangyongqiang0606,xchu42,zhang_shengjie,depeng1994,wqtshg_wt,peiyang,shengnan666,XuebinYang,sjtulxh,zhanj,zhujingjing,xuyafei,wangxiaotian995设为审查人
Z
zzq成员
7月24日 评论：

/compile

likedislike
Zzzq成员
7月24日 update merge request[project id: 8005834, iid: 1461, commit_id: 47191f58ad2818303296a8f7a288de1054dd1a3e] virtual merging success
CANN-robotCANN-robot成员
7月24日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月24日 评论：

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_checkpr ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
codecheck_codestyle ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>

[2026-07-24 10:59:27]    CI执行结束

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

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

任务名称状态日志下载链接
UT_Test_Python_superkernel ✅ SUCCESS >>>>>
ST_Test_Python_superkernel ✅ SUCCESS >>>>>
UT_Test_superkernel ✅ SUCCESS >>>>>
UT_Test_autofuse_framework ✅ SUCCESS >>>>>
ST_Test_autofuse_framework ✅ SUCCESS >>>>>
UT_Test_autofuse_ascendc_api ✅ SUCCESS >>>>>
ST_Test_autofuse_ascendc_api ✅ SUCCESS >>>>>
ST_Test_autofuse_e2e ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>

[2026-07-24 11:13:43]    CI执行结束

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

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-07-24 11:05:40]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月24日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
7月24日 添加了label：ci-pipeline-passed
atomgit-bot
atomgit-bot7月24日进行代码检视1
docs/env_install/tensorflow/env_tf.md
@@ -135,7 +135,7 @@ pip3 install "numpy==1.23.5" pandas decorator sympy scipy attrs psutil protobuf=
135135x86_64 架构可使用一键配置脚本自动完成上述步骤（在线安装 TF、NPU Adapter 及依赖）：
136136 
137137```bash
138-bash scripts/env_install/setup_tf_env.sh
138+bash scripts/env_install/tensorflow/setup_tf_env.sh
139139```
140140 
141141脚本完成后激活环境：
@@ -143,7 +143,7 @@
143143```bash
144144source scripts/env_install/env/activate_tf1.sh # TF 1.15
145145# 或
146146source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
atomgit-bot
atomgit-bot7月24日评论：

🟡 Medium Priority

与 README.md 相同的问题：脚本移动到 scripts/env_install/tensorflow/ 后默认 INSTALL_ROOT 改变，激活脚本实际生成在 scripts/env_install/tensorflow/env/activate_tf1.sh。本文档第 138 行已更新脚本运行命令，但第 144/146 行的激活命令仍指向旧路径 scripts/env_install/env/activate_tf1.sh，用户按文档操作将找不到文件。

建议：将第 144 行和第 146 行的激活路径从 scripts/env_install/env/activate_tf1.sh 改为 scripts/env_install/tensorflow/env/activate_tf1.sh，activate_tf2.sh 同理。

likedislike
atomgit-bot
atomgit-bot7月24日进行代码检视1
docs/env_install/tensorflow/env_tf.md
@@ -135,7 +135,7 @@ pip3 install "numpy==1.23.5" pandas decorator sympy scipy attrs psutil protobuf=
135135x86_64 架构可使用一键配置脚本自动完成上述步骤（在线安装 TF、NPU Adapter 及依赖）：
136136 
137137```bash
138-bash scripts/env_install/setup_tf_env.sh
138+bash scripts/env_install/tensorflow/setup_tf_env.sh
139139```
140140 
141141脚本完成后激活环境：
@@ -143,7 +143,7 @@
143143```bash
144144source scripts/env_install/env/activate_tf1.sh # TF 1.15
145145# 或
146146source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
atomgit-bot
atomgit-bot7月24日评论：

🟡 Medium Priority

与 autofuse/examples/tensorflow/README.md 中的问题相同。setup_tf_env.sh 移动到 scripts/env_install/tensorflow/ 后，脚本内 SCRIPT_DIR 解析为新路径，默认安装根目录变为 scripts/env_install/tensorflow/env/。

但 env_tf.md 第 144、146 行的激活命令仍指向旧路径：
用户按文档操作将遇到 "No such file or directory" 错误。

触发条件：用户使用默认路径运行一键脚本后，按此文档指引尝试激活环境。

建议：将激活脚本路径从 scripts/env_install/env/ 更新为 scripts/env_install/tensorflow/env/，与脚本移动后的默认安装位置保持一致。

改动建议
146
+ source scripts/env_install/tensorflow/env/activate_tf1.sh # TF 1.15
147
+ # 或
146
- source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
148
+ source scripts/env_install/tensorflow/env/activate_tf2.sh # TF 2.6.5
应用建议
likedislike
atomgit-bot
atomgit-bot7月24日进行代码检视1
autofuse/examples/tensorflow/README.md
@@ -22,7 +22,7 @@
22223. 也可使用一键配置脚本自动搭建环境（**仅 x86_64 架构可用**）：
2323 
2424 ```bash
25- bash scripts/env_install/setup_tf_env.sh
25+ bash scripts/env_install/tensorflow/setup_tf_env.sh
2626 ```
2727 
2828 脚本完成后激活环境：
@@ -30,7 +30,7 @@
3030 ```bash
3131 source scripts/env_install/env/activate_tf1.sh # TF 1.15
3232 # 或
3333 source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
atomgit-bot
atomgit-bot7月24日评论：

🟡 Medium Priority

变更将 setup_tf_env.sh 从 scripts/env_install/setup_tf_env.sh 移动到 scripts/env_install/tensorflow/setup_tf_env.sh。脚本内部通过 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" 动态计算安装根目录（第 25-26 行），因此移动后默认 INSTALL_ROOT 由 scripts/env_install/env/ 变为 scripts/env_install/tensorflow/env/。

但 README 第 31、33 行的激活命令仍指向旧路径：
用户按照文档执行 source scripts/env_install/env/activate_tf1.sh 将因文件不存在而失败（No such file or directory），因为脚本实际将环境安装到了 scripts/env_install/tensorflow/env/ 下。

触发条件：用户在不指定自定义安装路径（即使用默认路径）的情况下，按照 README 运行一键脚本后尝试激活环境。

建议：将激活脚本路径从 scripts/env_install/env/ 更新为 scripts/env_install/tensorflow/env/，与脚本移动后的默认安装位置保持一致。

改动建议
33
+ source scripts/env_install/tensorflow/env/activate_tf1.sh # TF 1.15
34
+ # 或
33
- source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
35
+ source scripts/env_install/tensorflow/env/activate_tf2.sh # TF 2.6.5
应用建议
likedislike
atomgit-bot
atomgit-bot7月24日进行代码检视1
autofuse/examples/tensorflow/README.md
@@ -22,7 +22,7 @@
22223. 也可使用一键配置脚本自动搭建环境（**仅 x86_64 架构可用**）：
2323 
2424 ```bash
25- bash scripts/env_install/setup_tf_env.sh
25+ bash scripts/env_install/tensorflow/setup_tf_env.sh
2626 ```
2727 
2828 脚本完成后激活环境：
@@ -30,7 +30,7 @@
3030 ```bash
3131 source scripts/env_install/env/activate_tf1.sh # TF 1.15
3232 # 或
3333 source scripts/env_install/env/activate_tf2.sh # TF 2.6.5
atomgit-bot
atomgit-bot7月24日评论：

🟡 Medium Priority

脚本从 scripts/env_install/setup_tf_env.sh 移动到 scripts/env_install/tensorflow/setup_tf_env.sh 后，SCRIPT_DIR 变为 scripts/env_install/tensorflow/，默认 INSTALL_ROOT 变为 scripts/env_install/tensorflow/env/，激活脚本实际生成在 scripts/env_install/tensorflow/env/activate_tf1.sh 和 scripts/env_install/tensorflow/env/activate_tf2.sh。

但 README 中仅更新了运行脚本的命令（第 25 行 bash scripts/env_install/tensorflow/setup_tf_env.sh），下方的激活命令（第 31/33 行 source scripts/env_install/env/activate_tf1.sh）未同步更新，仍指向旧默认路径。用户按文档操作将遇到 "No such file or directory" 错误。

建议：将第 31 行和第 33 行的激活路径从 scripts/env_install/env/activate_tf1.sh 改为 scripts/env_install/tensorflow/env/activate_tf1.sh，activate_tf2.sh 同理。

likedislike
Zzzq成员
7月24日 修改了pull request 的描述
此处折叠了5条事件消息 查看更多
xchu42xchu42成员
7月24日 解决了最后一个问题
xchu42
xchu42成员
7月24日 评论：

/lgtm

likedislike
X
xuyafei成员
7月24日 评论：

/lgtm

likedislike
CANN-robotCANN-robot成员
7月24日 添加了label：lgtm
Z
zhang_shengjie成员
7月24日 评论：

/lgtm

likedislike
zhanj
zhanj成员
7月25日 评论：

/approve

likedislike
CANN-robotCANN-robot成员
7月25日 添加了label：approved
CANN-robotCANN-robot成员
7月25日 合入了pull request