修复experimental脚本问题-ops-nn-AtomGit

Star946

Star946

已合并

修复experimental脚本问题 #951

yang-di52创建于 1月23日从yang-di52/ops-nn： bugfix1 合入到 cann/ops-nn： master

修复experimental脚本问题 #951

已合并

yang-di52创建于 1月23日

从yang-di52/ops-nn bugfix1合入到cann/ops-nn master

共 2 个文件变更+2-0

Mscripts/ci/check_experimental_example.sh+1-0

	@@ -15,6 +15,7 @@ pr_file=$(realpath "${1:-pr_filelist.txt}")
15	mkdir -p "${BUILD_PATH}"	15	mkdir -p "${BUILD_PATH}"
16	cd "${BUILD_PATH}" && rm -f CMakeCache.txt && cmake -DENABLE_EXPERIMENTAL=TRUE -DPREPROCESS_ONLY=ON ..	16	cd "${BUILD_PATH}" && rm -f CMakeCache.txt && cmake -DENABLE_EXPERIMENTAL=TRUE -DPREPROCESS_ONLY=ON ..
17		17
		18	+cd "${BASE_PATH}"
		CANN-robot1月23日代码逻辑和结构: 第18行添加的`cd "${BASE_PATH}"`命令是多余的重复代码。在代码第16行已经执行了`cd "${BUILD_PATH}"`进入构建目录，而在第28行（改动后的第29行）又再次执行了`cd "${BASE_PATH}"`。第18行的这个操作实际上在第28行会再次执行，导致不必要的目录切换操作。虽然这种重复在功能上不会导致错误，但它降低了代码的清晰度和可维护性，增加了不必要的操作。问题类型: 代码逻辑和结构文件路径: `scripts/ci/check_experimental_example.sh` 行号: 18 问题代码: `cd "${BASE_PATH}"` 修改建议: 建议删除第18行的`cd "${BASE_PATH}"`命令，因为第28行（改动后的第29行）已经包含了相同的操作。这样可以简化代码逻辑，避免不必要的目录切换。修改后，代码流程会更清晰：先进入构建目录执行cmake，然后解析文件，最后再回到基础目录执行后续操作。此评论由代码审查工具自动生成
18	{	19	{
19	result=$(python3 ${BASE_PATH}/scripts/util/parse_compile_changed_files.py ${pr_file} TRUE)	20	result=$(python3 ${BASE_PATH}/scripts/util/parse_compile_changed_files.py ${pr_file} TRUE)
20	} \|\| {	21	} \|\| {

Mscripts/ci/check_experimental_pkg.sh+1-0

	@@ -15,6 +15,7 @@ pr_file=$(realpath "${1:-pr_filelist.txt}")
15	mkdir -p "${BUILD_PATH}"	15	mkdir -p "${BUILD_PATH}"
16	cd "${BUILD_PATH}" && rm -f CMakeCache.txt && cmake -DENABLE_EXPERIMENTAL=TRUE -DPREPROCESS_ONLY=ON ..	16	cd "${BUILD_PATH}" && rm -f CMakeCache.txt && cmake -DENABLE_EXPERIMENTAL=TRUE -DPREPROCESS_ONLY=ON ..
17		17
		18	+cd "${BASE_PATH}"
18	{	19	{
19	result=$(python3 ${BASE_PATH}/scripts/util/parse_compile_changed_files.py ${pr_file} TRUE)	20	result=$(python3 ${BASE_PATH}/scripts/util/parse_compile_changed_files.py ${pr_file} TRUE)
20	} \|\| {	21	} \|\| {