已合并
[fix]all_reduce example添加 IsExampleEnabled() 守卫并补充绕过CMS验证方法 #3407
chenyizhuo创建于 21 天前
[fix]all_reduce example添加 IsExampleEnabled() 守卫并补充绕过CMS验证方法 #3407
已合并
chenyizhuo创建于 21 天前
chenyizhuo
21 天前

描述

当前自编包无法通过CMS验证,因此在文档中加入了使用npu-smi工具绕过CMS验签的方法。
experimental目录要求实验特性加入 IsExampleEnabled() 守卫,因此本次修改加入了该函数。

关联的Issue

issue#784

测试

本地checker测试通过。

文档更新

更新了hccl/experimental/ops/all_reduce/example/README.md及hccl/experimental/ops/all_reduce/example/README_en.md。添加了 IsExampleEnabled() 相关的说明及绕过CMS验证的方法。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 chenyizhuo 的贡献)
Cchenyizhuo
21 天前 创建了 pull request,commit 99a6d0e2
atomgit-bot
atomgit-bot
21 天前 评论:

变更摘要

本次变更针对 hccl/experimental/ops/all_reduce/example 实验特性示例,为 InsV2AllReduceExperimentalSoleExecutor 增加 IsExampleEnabled() 守卫,使其在实验特性未启用时跳过算法选择;同时在文档中补充了通过 npu-smi 工具绕过 CMS 验签的说明,以解决自编包无法通过 CMS 验证的问题。

主要改动

  • 新增 IsExampleEnabled() 守卫方法:在 ins_v2_all_reduce_experimental_sole_executor.h 中声明、并在同名 .cc 中实现 IsExampleEnabled(),通过编译期常量 exampleEnabled 与环境变量 HCCL_EXPERIMENTAL_EXAMPLE(需为 true)共同判断实验示例是否启用。
  • CalcCostCoeff 增加启用判断:在 InsV2AllReduceExperimentalSoleExecutor::CalcCostCoeff 入口处调用 IsExampleEnabled(),未启用时输出 HCCL_WARNING 并返回空结果 {},从而在算法选择中跳过该示例算法。
  • 补充绕过 CMS 验签文档:更新 all_reduce/example 下的 README.md 与 README_en.md,加入使用 npu-smi 工具绕过 CMS 验签的方法以及 IsExampleEnabled() 相关说明。
likedislike
不准确?
atomgit-bot
atomgit-bot
21 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
21 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
21 天前 评论:

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)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了109条消息 查看更多
严正行严正行成员
18 天前 解决了最后一个问题
chenyia成员
18 天前 评论:

/lgtm

likedislike
philo
philo成员
18 天前 评论:

/approve

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