已合并
算子插件迁移 #8474
welt_lester创建于 8月10日
算子插件迁移 #8474
已合并
welt_lester创建于 8月10日
welt_lester
welt_lester
8月10日

描述

落实issue #4279,将adaptive_avg_pool2d、adaptive_max_pool2d、ascend_anti_quant、ascend_dequant、ascend_quant、batch_mat_mul_v3、batch_norm、celu_v2、instance_norm_v3、ascend_quant、log_softmax_v2、mat_mul_v3、unique_with_counts_and_sorting这13个算子对应的插件从common\src\framework迁出到各自算子目录新建的\framework目录下。

关联的Issue

[Bug-Report|缺陷反馈]: 有对应算子的插件不应该放到公共目录 #4279

测试

compile、本地验证build通过

文档更新

不涉及

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 welt_lester 的贡献)
welt_lesterwelt_lester
8月10日 创建了 pull request,commit e9fa1f77
atomgit-bot
atomgit-bot
8月10日 评论:

变更摘要

此 PR 将 13 个 ONNX 算子插件文件迁移到按算子类别分层的目录结构中(如 activation/、index/、matmul/、norm/、pooling/、quant/ 等),使每个插件文件归入其对应算子的子目录下。除行尾换行符从 CRLF 统一为 LF 外,所有文件内容无任何功能性变更,属于纯粹的目录重组操作。

主要改动

  • 算子插件按类别归入分层目录:将 celu_onnx_plugin.cpp、logsoftmax_onnx_plugin.cpp、unique_onnx_plugin.cpp、batch_matmul_onnx_plugin.cpp、matmul_onnx_plugin.cpp、batch_normalization_onnx_plugin.cpp、instancenormalization_onnx_plugin.cpp、adaptive_average_pool2d_onnx_plugin.cpp、adaptive_max_pool2d_onnx_plugin.cpp、ascend_anti_quant_onnx_plugin.cpp、ascend_dequant_onnx_plugin.cpp、ascend_quant_onnx_plugin.cpp、int8_quantize_onnx_plugin.cpp 共 13 个文件分别迁移至 activation/、index/、matmul/、norm/、pooling/、quant/ 等类别目录下对应的算子子目录中。

  • 行尾换行符统一规范化:所有迁移文件的换行符从 CRLF(\r\n)统一转换为 LF(\n),代码逻辑、注册的算子类型(REGISTER_CUSTOM_OP)、参数解析函数(如 ParseParamsCelu、ParseParamsLogSoftmax 等)及框架类型(ONNX)均保持不变。

likedislike
不准确?
atomgit-bot
atomgit-bot
8月10日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
8月10日 添加了label:cann-cla/no
CANN-robot
CANN-robot成员
8月10日 评论:

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.
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
activation ✅ 苏跃明, 唐玮玮 (2/2) ✅ 苏跃明 (1/1)
index ✅ 苏跃明, 唐玮玮 (2/2) ✅ 苏跃明 (1/1)
matmul ✅ 唐玮玮, 徐炳麟 (2/2) ✅ 唐玮玮, 徐炳麟 (2/1)
norm ✅ 苏跃明, 唐玮玮 (2/2) ✅ 苏跃明 (1/1)
pooling ✅ 苏跃明, 唐玮玮 (2/2) ✅ 苏跃明 (1/1)
quant ✅ 苏跃明, 徐炳麟, 唐玮玮 (3/2) ✅ 苏跃明, 徐炳麟 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了67条消息 查看更多
CANN-robotCANN-robot成员
8月17日 添加了label:approved
xubinglin成员
8月17日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8月17日 添加了label:lgtm
CANN-robotCANN-robot成员
8月17日 关闭了关联的issue
CANN-robotCANN-robot成员
8月17日 合入了pull request