已合并
fix:修复HCCL软件目录重构后遗留问题 #2945
fix:修复HCCL软件目录重构后遗留问题 #2945
已合并
miaoziyi1031创建于 9月7日
miaoziyi1031
miaoziyi1031
9月7日

描述

本 PR 用于处理前期 HCCL 软件目录与软件架构保持一致目录重构后的遗留问题,主要涉及 physical_level 模块职责归位及 experimental 文档路径同步。
前置已合并pr:https://gitcode.com/cann/hccl/pull/2823

具体修改如下:
1、调整 physical_level 相关模块目录归属
physical_level 模块通过 HCOMM Rank Graph 接口获取物理拓扑相关信息,经归一化、校验后构建 topoInfo->physicalLevels,属于拓扑信息的获取、转换与初始化能力。
原文件位于:src/ops/op_common/algorithm/topo_match/
本次将以下 3 个文件迁移至:src/ops/op_common/topo_info/
涉及文件:physical_level.h、physical_level_build.cc、physical_level_normalize.cc
三个文件均为纯目录迁移,源码内容不变。

2、同步调整 CMake 构建归属
将:physical_level_build.cc、physical_level_normalize.cc
从:src/ops/op_common/algorithm/topo_match/CMakeLists.txt的源文件列表中移除,
并加入:src/ops/op_common/topo_info/CMakeLists.txt
保证 physical_level 模块随职责归位后由 topo_info 模块统一管理,两个源文件仍各编译一次,编译目标及链接关系不变。

3、修复 experimental 文档中的旧 src 路径
更新以下中英文 README:
experimental/ops/all_reduce/example/README.md
experimental/ops/all_reduce/example/README_en.md

两个文档各修复 6 处目录重构后未同步更新的 src 旧路径,包括:
executor/
→ algorithm/executor/

template/
→ algorithm/template/

op_common/executor/
→ op_common/algorithm/executor/

op_common/template/
→ op_common/algorithm/template/

仅修正指向 src 主链路的路径引用,未调整 experimental 自身现有目录结构,也未修改 src/ops/op_common/selector/ 等未发生迁移的路径。

关联的Issue

https://gitcode.com/cann/hccl/issues/658

测试

ST全量通过、已完成Checker自验证

文档更新

更新以下文档:
experimental/ops/all_reduce/example/README.md
experimental/ops/all_reduce/example/README_en.md
分别修复 6 处目录重构后未同步更新的 src 路径引用,共 12 处。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 miaoziyi1031 的贡献)
miaoziyi1031miaoziyi1031
9月7日 创建了 pull request,commit 63cc8bf0
miaoziyi1031miaoziyi1031
9月7日 关联了issue:[Requirement|需求建议]: HCCL仓软件目录与软件架构保持一致(src/ops目录结构对齐)
atomgit-bot
atomgit-bot
9月7日 评论:

变更摘要

本 PR 用于修复 HCCL 软件目录重构(与软件架构保持一致)后遗留的目录归属与路径引用问题:将 physical_level 模块(通过 HCOMM Rank Graph 接口获取物理拓扑信息并构建 topoInfo->physicalLevels,属于拓扑信息获取、转换与初始化能力)从 src/ops/op_common/algorithm/topo_match/ 归位迁移至 src/ops/op_common/topo_info/,同步调整 CMake 构建归属,并修复 experimental 中英文 README 中未随重构同步更新的旧 src 路径引用。涉及 3 个源文件纯目录迁移(内容不变)、2 个 CMakeLists 的源文件归属调整及 2 个文档的路径修正。

主要改动

  • physical_level 模块文件目录迁移:将 physical_level.h、physical_level_build.cc、physical_level_normalize.cc 三个文件从 src/ops/op_common/algorithm/topo_match/ 迁至 src/ops/op_common/topo_info/,为纯目录迁移,源码内容不变,使模块随职责归位后由 topo_info 统一管理。
  • CMake 构建归属同步调整:将 physical_level_build.cc、physical_level_normalize.cc 从 src/ops/op_common/algorithm/topo_match/CMakeLists.txt 的源文件列表中移除,并加入 src/ops/op_common/topo_info/CMakeLists.txt,两个源文件仍各编译一次,编译目标与链接关系不变。
  • experimental 文档旧路径修复:更新 experimental/ops/all_reduce/example/README.md 与 README_en.md,各修复 6 处(共 12 处)重构后未同步的 src 路径引用,包括 executor/ → algorithm/executor/、template/ → algorithm/template/、op_common/executor/ → op_common/algorithm/executor/、op_common/template/ → op_common/algorithm/template/,仅修正主链路路径,未改动 experimental 自身目录结构与 selector/ 等未迁移路径。
likedislike
不准确?
atomgit-bot
atomgit-bot
9月7日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
9月7日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月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
experimental ✅ 颜业峰, 严正行 (2/2) ✅ 颜业峰 (1/1)
src ✅ 李标智, 颜业峰, 严正行 (3/2) ✅ 李标智, 颜业峰 (2/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成员
9月7日 评论:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
9月7日 将adum,leabclove,weixin_43960572,yanglianxiang2,linyf950,yanzhenghang,chenliang267,yanyefeng,zhongyu1,chenhao_,songmingyang,wenxuemin,yangshaohua,bz_lee,chenyia,zhangxp1030,silencesirius设为评审人
CANN-robotCANN-robot成员
9月7日 将leabclove,yanyefeng,wenxuemin,bz_lee设为审查人
miaoziyi1031
miaoziyi1031
9月7日 评论:

/compile

likedislike
miaoziyi1031miaoziyi1031
9月7日 预合并成功(commit_id: 39ffac48205c16bab31e003841abacd72e087f86)
CANN-robot
CANN-robot成员
9月7日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月7日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月7日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月7日 添加了label:ci-pipeline-passed
严正行
严正行成员
9月7日 评论:

/lgtm

likedislike
bz_lee成员
9月8日 评论:

/approve

likedislike
philo
philo成员
9月8日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月8日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
9月8日 合入了pull request