已合并
【PR】bugfix: reduce算子融合case,1)针对ATT与Codegen校验merge_size问题统一取向量化轴repeats 2)针对codegen生成代码时2个broadcast算子消费同一父节点导致生成代码变量名冲突问题进行兼容解决 #2156
Nexen_NiSong创建于 28 天前
【PR】bugfix: reduce算子融合case,1)针对ATT与Codegen校验merge_size问题统一取向量化轴repeats 2)针对codegen生成代码时2个broadcast算子消费同一父节点导致生成代码变量名冲突问题进行兼容解决 #2156
已合并
Nexen_NiSong创建于 28 天前
Nexen_NiSong
Nexen_NiSong成员
28 天前

Pull Request

描述

【PR】bugfix: reduce算子融合case,1)针对ATT与Codegen校验merge_size问题统一取向量化轴repeats 2)针对codegen生成代码时2个broadcast算子消费同一父节点导致生成代码变量名冲突问题进行兼容解决

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:

  1. 通过CI平台构建了AF包,上传到环境上进行了验证,明确了此前失败的用例已执行通过;
  2. 针对其他此前执行成功的用例再新AF包中进行了验证,修改代码后仍然成功
  3. 已通过gitcode平台CI流水线,UT执行通过
  4. 针对修改的代码,已拉通相关模块熟悉的责任人明确修改无误且不引入额外问题

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Nexen_NiSong 的贡献)
Nexen_NiSongNexen_NiSong成员
28 天前 添加了label:enhancement
Nexen_NiSongNexen_NiSong成员
28 天前 创建了 pull request,commit df44384c
CANN-robot
CANN-robot成员
28 天前 评论:

Hi @tourse, welcome to submitting your first PR to graph-autofusion!

PR Merge Steps

1. CLA Signing

If the current PR label includes cann-cla/yes, it means you have signed the CLA and can proceed to the next step. If the label includes cann-cla/no, please sign the CLA first. If you have any questions, please refer to the FAQ.

2. CI Check

Please comment /compile to trigger the CI pipeline check. If the CI run is successful, the PR will be tagged with ci-pipeline-passed and you can proceed to the next step. If the CI run fails, the PR will be tagged with ci-pipeline-failed, please check the CI logs to fix the issues in the PR. If you have any questions, please refer to the FAQ.

3. Code Review

After CI passes, please refer to the PR Approval Progress and proactively @ the committers in the table to review the code. After approval, committers will comment /lgtm and /approve. Once the lgtm and approved labels are successfully added, the PR will be merged automatically.

likedislike
CANN-robotCANN-robot成员
28 天前 添加了label:cann-cla/no
CANN-robot
CANN-robot成员
28 天前 评论:

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
repo-cann/graph-autofusion ✅ wangxiaotian995, yangyongqiang0606, liyuewei (3/2) ✅ yangyongqiang0606 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了77条消息 查看更多
liyuewei
liyuewei成员
25 天前 评论:

/lgtm

likedislike
Nexen_NiSongNexen_NiSong成员
25 天前 修改了pull request 的描述
yangyongqiang
yangyongqiang成员
25 天前 评论:

/approve

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