已关闭
revert: 回退 scatter_reduce 相关 4 个 PR 以消除性能影响(9.2.0-beta.2) #10043
BartonFang创建于 9月8日关闭于 9月23日
revert: 回退 scatter_reduce 相关 4 个 PR 以消除性能影响(9.2.0-beta.2) #10043
已关闭
BartonFang创建于 9月8日关闭于 9月23日
BartonFang
BartonFang成员
9月8日

背景

在 9.2.0-beta.2 基础上,以下 4 个合入 master 的 PR 疑似引入 scatter_elements/scatter_reduce 主路径性能回退,需整体回退验证:

  • #6460 Add min, max, and mean modes to the scatter_reduce operator(d382188b4)
  • #6624 fix precision bug: relax host-side cache-op xDim1 guard(987b2195c)
  • #7736 Add interception for AICPU of the newly added mode(6cb151633)
  • #8714 Restore scatter deterministic multicore docs(27d676070)

修改内容

在 9.2.0-beta.2(c43952d5)基础上依次 revert 上述 4 个合入提交(逆序 8714→7736→6624→6460),共 4 个 revert commit:

  • Revert "Restore scatter deterministic multicore docs"
  • Revert "Add interception for AICPU of the newly added mode"
  • Revert "fix precision bug:relax host-side cache-op xDim1 guard"
  • Revert "Add min, max, and mean modes to the scatter_reduce operator"

净变更:30 个文件,删除约 7700 行(含新增的 scatter_reduce 相关测试/文档/示例),scatter_elements_v2 主路径恢复至 6460 合入前状态。回退过程中对与后续提交叠加的少量冲突(scatter_elements.cpp 常量恢复、格式类差异)按“仅撤销目标 PR 语义、保留 9.2.0-beta.2 其它提交改动”原则处理,经验证核心文件与 6460 合入前逐字一致。

测试

  • 已核对回退后 scatter_elements_v2 无 6460 新增 min/max/mean reduce 分支残留,新增文件全部删除。
  • 建议在 9.2.0-beta.2 环境对 scatter_reduce/scatter_elements 相关用例与性能基线复测(编译验证待 CI/环境执行)。
likedislike
当前Pull Request已关闭, 关闭人@BartonFang
BartonFangBartonFang成员
9月8日 创建了 pull request,commit 8ad5dc01
atomgit-bot
atomgit-bot
9月8日 评论:

🤖 正在生成合并请求摘要,请稍候…

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

🤖 AI 代码检视正在进行中,请稍候…

likedislike
不准确?
CANN-robotCANN-robot成员
9月8日 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
9月8日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月8日 评论:

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

⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)

Module Approval Details

module lgtm status approve status
*/*/docs/acl*.md ❌ (0/2)(You can also ask: 王瑞, 冯英, 蔡雯雯, 陈娇, 胡碧霞) ❌ (0/1)(You can also ask: 蔡雯雯, 杨露, 陈娇, 杜慧萍, 冯英)
*/*/op_api/*.h ❌ (0/2)(You can also ask: 苏跃明, 黄迪, 杨阳, 王瑞, 钱泽洪) ❌ (0/1)(You can also ask: 王永光, 汤磊)
*/*/op_graph/*_proto.h ❌ (0/2)(You can also ask: 唐玮玮, 苏跃明, 胡碧霞, 王瑞, 曾娟) ❌ (0/1)(You can also ask: 汤磊, 王永光)
*/*/op_host/*_def.cpp ❌ (0/2)(You can also ask: 刘波, 唐玮玮, 陈琦, 王永光, 曾娟) ❌ (0/1)(You can also ask: 王永光, 汤磊)
docs ❌ (0/2)(You can also ask: 曾娟, 杨露, 唐玮玮, 汤平川, 杨阳) ❌ (0/1)(You can also ask: 冯英, 蔡雯雯, 杨露, 陈娇, 杜慧萍)
index ❌ (0/2)(You can also ask: 苏跃明, 杨阳, 陈风, 汤平川, 唐超) ❌ (0/1)(You can also ask: 王瑞, 黄迪, 王星, 汤平川, 唐燕峰)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
9月8日 将zengjuan,chenfeng61,chaotang233,kevin_huang1234,FelixTang7,crystalhu,yangyang016,fanqirui,gitee-duhuiping,wangrui_,su-yueming,tang-lei01,chenqi317,wang-xing001,liubo75,tangweiwei2,gitcode-chenjiao,chenxingyu18,pingchuantang,qianzehong,wangyongguang,fengying555682,caiwenwen,yanglu-1设为评审人
CANN-robotCANN-robot成员
9月8日 将chenfeng61,kevin_huang1234,FelixTang7,gitee-duhuiping,wangrui_,su-yueming,tang-lei01,wang-xing001,gitcode-chenjiao,chenxingyu18,pingchuantang,qianzehong,wangyongguang,fengying555682,caiwenwen,yanglu-1设为审查人
atomgit-bot
atomgit-bot
9月8日 评论:

⚠️ 本次变更过大(28 个可审文件、8464 行,预估需 8 轮分片审查),已超出 AI 代码评审的处理范围,本次跳过。建议拆分为更小的 PR 以获得有效评审。

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

⚠️ 本次变更过大(28 个可审文件、8464 行,预估需 8 轮分片审查),已超出 AI 代码评审的处理范围,本次跳过。建议拆分为更小的 PR 以获得有效评审。

likedislike
不准确?
yuning_chen
yuning_chen成员
9月8日 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike
BartonFangBartonFang成员
9月8日 预合并成功(commit_id: 622c9d1ddaf9dbf546ef29f17426b23345ccdb7c)
BartonFangBartonFang成员
9月8日 强制推送  1 个提交:075c9892-Revert "Add min, max, and mean modes to the scatter_reduce operator"
BartonFangBartonFang成员
9月8日 预合并成功(commit_id: 118765b472c5aa90a16749a7ffe3cf21d3d965e2)
CANN-robotCANN-robot成员
9月8日 删除了label:cann-cla/yes
CANN-robotCANN-robot成员
9月8日 添加了label:cann-cla/yes
yuning_chen
yuning_chen成员
9月8日 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike
BartonFangBartonFang成员
9月8日 预合并成功(commit_id: bf5789aa70a8125ec518f0d4949a68cec406ba48)
BartonFangBartonFang成员
9月8日 强制推送  2 个提交:8b16d501-Revert "Add min, max, and mean modes to the scatter_reduce operator",f6877279-fix: 6460 回退后补回 9.2.0-beta.2 上其它提交的改动
BartonFangBartonFang成员
9月8日 预合并成功(commit_id: f0917834c7ab73cb23466c3513e83cd82089aef0)
CANN-robotCANN-robot成员
9月8日 删除了label:cann-cla/yes
CANN-robotCANN-robot成员
9月8日 添加了label:cann-cla/yes
BartonFangBartonFang成员
9月23日 关闭了 pull request