已合并
delete quant_all_reduce & quant_reduce_scatter aclnn #12003
yifuxiong创建于 22 天前
delete quant_all_reduce & quant_reduce_scatter aclnn #12003
已合并
yifuxiong创建于 22 天前
从已删除 :pr_920_beta2合入到cann/ops-transformer9.2.0-beta.2
yifuxiong
yifuxiong成员
22 天前

描述

删除quant_all_reduce和quant_reduce_scatter算子aclnn接口实现

关联的Issue

https://gitcode.com/cann/ops-transformer/issues/3782

测试

不涉及

文档更新

更新了README.md文件

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yifuxiong 的贡献)
yifuxiongyifuxiong成员
22 天前 创建了 pull request,commit 33c84b5a
yifuxiongyifuxiong成员
22 天前 关联了issue:[Requirement|需求建议]:
atomgit-bot
atomgit-bot
22 天前 评论:

变更摘要

该 PR 根据关联 Issue(#3782)删除 quant_all_reduce 与 quant_reduce_scatter 两个算子的 aclnn 接口实现。mc2/quant_all_reduce/op_api/aclnn_quant_all_reduce.cpp 和 mc2/quant_reduce_scatter/op_api/aclnn_quant_reduce_scatter.cpp 中原有的参数校验(空指针、K-G/MX 量化 dtype 支持、format、group 长度等检查)、内部接口声明及 NnopbaseSetHcclServerType 调用被整体删除,公开入口 aclnnQuantAllReduceGetWorkspaceSize/aclnnQuantAllReduce 与 aclnnQuantReduceScatterGetWorkspaceSize/aclnnQuantReduceScatter 的函数体被改为直接返回 ACLNN_SUCCESS;同时删除对应的示例程序和 UT 测试,并更新了 README.md。

主要改动

  • aclnn 接口实现置空:在 aclnn_quant_all_reduce.cpp 与 aclnn_quant_reduce_scatter.cpp 中删除全部参数校验函数(如 QuantAllReduceCheckParams、CheckParams 及各 dtype/format 检查)、内部接口声明(aclnnInnerQuantAllReduceGetWorkspaceSize、aclnnInnerQuantReduceScatterGetWorkspaceSize 等)和 NnopbaseSetHcclServerType 调用,两个公开入口接口均改为直接 return ACLNN_SUCCESS;,不再执行实际算子逻辑。
  • 删除示例程序:删除 mc2/quant_all_reduce/examples/test_aclnn_quant_all_reduce.cpp 与 mc2/quant_reduce_scatter/examples/test_aclnn_quant_reduce_scatter.cpp 两个多设备(多线程)aclnn 调用示例,其中包含 aclrt 资源管理、HcclComm 通信域初始化及两阶段接口调用流程。
  • 删除 UT 测试:删除 mc2/quant_all_reduce/tests/ut/op_api/test_aclnn_quant_all_reduce.cpp 的全部测试用例(dtype 正常/异常、私有 format、group 长度边界、空指针等用例);mc2/quant_reduce_scatter/tests/ut/op_api/test_aclnn_quant_reduce_scatter.cpp 中的测试用例同样被移除,仅保留文件头注释。
  • 文档更新:根据 PR 描述同步更新了 README.md,移除与上述两个算子 aclnn 接口相关的内容。
likedislike
不准确?
CANN-robotCANN-robot成员
22 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
22 天前 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
22 天前 评论:

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
*/*/README.md ✅ 蒋礼锐, wang-minbo, libohao6, 蔡雯雯, chenjunjian11 (5/2) ✅ 蔡雯雯 (1/1)
*/*/docs/acl*.md ✅ 蔡雯雯, chenjunjian11, 蒋礼锐, wang-minbo, libohao6 (5/2) ✅ 蔡雯雯 (1/1)
docs ✅ 蒋礼锐, wang-minbo (2/2) ✅ 蒋礼锐, wang-minbo (2/1)
mc2 ✅ wang-minbo, libohao6, 蒋礼锐, chenjunjian11 (4/2) ✅ 蒋礼锐, wang-minbo, libohao6 (3/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
22 天前 将captainmiaow,monologue815,yu-xinjie62,jiang-lirui,Allan_Yu,gitee-duhuiping,wang-minbo,claire8090,tgwsakiko_,xtqh,songkai111,liudan12,chenjunjian11,yangzeheng,liuboxi,gitcode-chenjiao,luobaiqing,zhong-zixin,Ww7012,libohao6,mabing1118,macech,fengying555682,caiwenwen,yanglu-1设为评审人
CANN-robotCANN-robot成员
22 天前 将captainmiaow,monologue815,yu-xinjie62,jiang-lirui,Allan_Yu,gitee-duhuiping,wang-minbo,claire8090,tgwsakiko_,xtqh,songkai111,liudan12,chenjunjian11,yangzeheng,liuboxi,gitcode-chenjiao,luobaiqing,zhong-zixin,Ww7012,libohao6,mabing1118,macech,fengying555682,caiwenwen,yanglu-1设为审查人
yifuxiong
yifuxiong成员
22 天前 评论:

compile

likedislike
yifuxiongyifuxiong成员
22 天前 预合并成功(commit_id: 6c428ba135b5ca722b5968048b20f4ac09cf8f2b)
CANN-robot
CANN-robot成员
22 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
22 天前 添加了label:ci-pipeline-running
此处折叠了6条事件消息 查看更多
CANN-robotCANN-robot成员
21 天前 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
21 天前 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

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

compile

likedislike
yifuxiongyifuxiong成员
21 天前 预合并成功(commit_id: 6a49a0558587da81f1c78b460da4f9e63c96331e)
CANN-robot
CANN-robot成员
21 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
21 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
21 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
21 天前 添加了label:ci-pipeline-passed
Beau Howard
Beau Howard成员
21 天前 评论:

/lgtm
/approve

likedislike
chenjianjun11成员
21 天前 评论:

/lgtm

likedislike
jiang-lirui成员
21 天前 评论:

/approve
/lgtm

likedislike
wang-minbo成员
21 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
21 天前 添加了label:lgtm
caiwenwen成员
21 天前 评论:

/lgtm
/approve

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