已合并
revert aicpu of scatter_elements_v2 #6574
青青吾心创建于 6月25日
revert aicpu of scatter_elements_v2 #6574
已合并
青青吾心创建于 6月25日
青青吾心
青青吾心
6月25日

描述

回退scatter_elements_v2A算子的AICPU新增代码逻辑,保证顺利出包

关联的Issue

https://gitcode.com/cann/ops-nn/issues/2654

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 青青吾心 的贡献)
青青吾心青青吾心
6月25日 创建了 pull request,commit 5ed2e966
CANN-robotCANN-robot成员
6月25日 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
6月25日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月25日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
6月25日 评论:

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


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
index 唐燕峰, 唐玮玮 (2/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

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

likedislike
青青吾心青青吾心
6月25日 update merge request[project id: 7665709, iid: 6574, commit_id: c4f309dde0282475e992693f4e5aa550681cb6e5] virtual merging success
青青吾心青青吾心
6月25日 强制推送  101 个提交:8d8e4a55-100 commits from branch mastercddeeafc-revert aicpu of scatter_elements_v2
青青吾心青青吾心
6月25日 update merge request[project id: 7665709, iid: 6574, commit_id: a3d07385e1d88dd0af9a7a64ab541a9f7c55166e] virtual merging success
CANN-robotCANN-robot成员
6月25日 删除了label:stat/needs-squash
CANN-robot
CANN-robot成员
6月25日 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): stat/needs-squash.

likedislike
青青吾心青青吾心
6月25日 update merge request[project id: 7665709, iid: 6574, commit_id: b57ee276cc001a4b8784f73fb79637eaaa39d8e1] virtual merging success
青青吾心青青吾心
6月25日 强制推送  1 个提交:91640574-revert aicpu of scatter_elements_v2
青青吾心青青吾心
6月25日 update merge request[project id: 7665709, iid: 6574, commit_id: 8e2b35a493c4dab64630ac6746b64c8a5491b526] virtual merging success
青青吾心
青青吾心
6月25日 评论:

compile

likedislike
青青吾心青青吾心
6月25日 update merge request[project id: 7665709, iid: 6574, commit_id: 3e7397b6d167c6c12fe673ccaa892e56e52357dc] virtual merging success
CANN-robotCANN-robot成员
6月25日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月25日 评论:

流水线任务触发成功
任务链接 [642dfe9614dd4bc3a525bd5f48846b1a][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950 ✅ SUCCESS >>>>> >>>>>
Compile_Pre ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_classify ✅ SUCCESS >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
UT_Test_opgraph ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>
UT_Test_report_lcov ⚠️ WARNING >>>>>

[2026-06-25 21:39:27]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月25日 评论:

流水线任务触发成功
任务链接 [87ebfb3fbbf14abb8216f92987f3398a][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_checkpr ✅ SUCCESS
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
SCA ✅ SUCCESS >>>>>

[2026-06-25 21:14:11]    CI执行结束

likedislike
atomgit-bot
atomgit-bot
6月25日 评论:

代码审查

审查总结

逐文件审查结果:

文件 审查结论
index/scatter_elements/op_kernel_aicpu/CMakeLists.txt 无问题 — 移除 v2 专属的依赖检查和 target 跟踪,恢复为与其他所有 AICPU op 一致的标准模式
index/scatter_elements/op_kernel_aicpu/scatter_elements_aicpu.cpp 1 个 P3 建议(<memory> 头文件移除)
index/scatter_elements/op_kernel_aicpu/scatter_elements_aicpu_def.cpp 无问题 — 移除 v2 新增的 axis/reduction/include_self 属性和 OPEN_OPS_FLAG
index/scatter_elements/tests/ut/op_kernel_aicpu/test_scatter_elements.cpp 无问题 — 移除了与已移除功能对应的测试用例
index/scatter_elements_v2/tests/st/aclnnScatterReduce/atk_aclnnScatterReduce.json 无问题 — 仅 +1/-1 微小变更,无法检测到具体问题

按优先级统计: P0: 0, P1: 0, P2: 0, P3: 1

整体风险评估:低风险。 这是一个干净的回退(revert),将 scatter_elements AICPU kernel 从 v2 版本(支持 max/min/mean/include_self 等归约操作)回退到原始的简化版本(仅支持 none/add/mul)。CMakeLists.txt 恢复了与其他 op 一致的标准模式;核心计算逻辑正确地将 reduction 简化到三种基本操作;op 定义文件恢复为 pre-v2 状态;测试文件正确移除了不再适用的测试用例。唯一的小问题是 #include <memory> 被移除但 std::shared_ptr 仍被直接使用,依赖传递包含,属于代码卫生问题。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
atomgit-bot
atomgit-bot
6月25日 评论:

🤖 AI Code Review

⚠️ 自定义模型 Provider 调用失败

本次审查使用的自定义模型 Provider 无法正常调用,已停止重试。

请检查 Provider 配置:

  • Base URL 是否正确、网络是否可达
  • API Key 是否有效
  • 模型名称是否存在

修正配置后,重新触发审查即可。

追踪 ID: task-21043-run-19470

likedislike
CANN-robotCANN-robot成员
6月25日 添加了label:api-check-pass
CANN-robotCANN-robot成员
6月25日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月25日 添加了label:ci-pipeline-passed
FelixTang7成员
6月25日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
6月25日 添加了label:approved
青青吾心青青吾心
6月25日 修改了pull request 的描述
青青吾心青青吾心
6月25日 修改了pull request 的描述
青青吾心青青吾心
6月25日 修改了pull request 的描述
tangweiwei2成员
6月25日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
6月25日 添加了label:lgtm
CANN-robotCANN-robot成员
6月25日 合入了pull request