已合并
[inductor][refactor] move npu_fusion_attention_graph to fx_passes/pattern_match #33269
jimmycao9929创建于 4月7日
[inductor][refactor] move npu_fusion_attention_graph to fx_passes/pattern_match #33269
已合并
J
jimmycao9929
4月7日
4月7日
# 【合入来源】
- [x] 重构优化
# 【修改方案】
把npu_fusion_attention_graph.py属于图优化，因此，将其移到fx_passes/pattern_match下
# 【资料变更】
不涉及
# 【接口变更】
不涉及
# 【功能验证】
单元测试
# 【CheckList】
- [x] 代码注释完备，正确记录错误日志
- [x] 代码实现进行了返回值、空指针等校验
- [x] PR标题正确使用类型标签，如：feat、fix、refactor、docs、test等
- [x] PR持续集成流水线（CI）执行通过，代码检查无异常
Pull Request已成功合入, 合并人@ascend-robot
（感谢 jimmycao9929 的贡献）4月7日 创建了 pull request，commit 3d8f1b1f
ascend-robot
4月7日 评论：
4月7日 评论：
ascend-robot
4月7日 评论：
4月7日 评论：
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 |
|---|---|---|
| test | ✅ weizhan4, crazyDannyBoy, rain-666 (3/2) | ✅ crazyDannyBoy (1/1) |
| torch_npu/_inductor | ✅ weizhan4, crazyDannyBoy, rain-666 (3/2) | ✅ crazyDannyBoy (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
jimmycao9929, thanks for your pull request. All authors of the commits have signed the CLA. 👍
4月7日 添加了label：ascend-cla/yes
jimmycao9929
4月7日 评论：
4月7日 评论：
compile
4月7日 添加了label：ci-pipeline-running
此处折叠了9条事件消息 查看更多
4月8日 添加了label：ci-pipeline-failed
ascend-robot
4月8日 评论：
4月8日 评论：
流水线 PR-pipeline_pytorch#14581 （重试第3次）运行失败
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| Build_ARM_inductor | ✅ | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | ❌ | >>> |
| UT_X86_Part_02 | ✅ | >>> | |
| UT_ARM_A3_Part_01 | ✅ | >>> | |
| UT_ARM_A3_Part_02 | ✅ | >>> | |
| UT_DIST_X86_Part_01 | 🛑 | >>> | |
| UT_DIST_X86_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_03 | 🛑 | >>> | |
| UT_DIST_X86_Part_04 | 🛑 | >>> | |
| UT_inductor_Part_01 | ✅ | >>> | |
| UT_inductor_Part_02 | ✅ | >>> | |
| UT_inductor_Part_03 | ✅ | >>> | |
| UT_inductor_Part_04 | ✅ | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ❌ | >>> |
Jjimmycao9929
4月8日 修改标题为 “[inductor][refactor] move npu_fusion_attention_graph to fx_passes/pattern_match”，原标题为“[inductor][refactor] move npu_fusion_attention_graph.py to fx_passes/pattern_match”
4月8日 修改标题为 “[inductor][refactor] move npu_fusion_attention_graph to fx_passes/pattern_match”，原标题为“[inductor][refactor] move npu_fusion_attention_graph.py to fx_passes/pattern_match”
此处折叠了5条事件消息 查看更多
4月8日 添加了label：ci-pipeline-failed
jimmycao9929
4月8日 评论：
4月8日 评论：
compile
4月8日 删除了label：ci-pipeline-failed
4月8日 添加了label：ci-pipeline-running
4月8日 删除了label：ci-pipeline-running
4月8日 添加了label：ci-pipeline-passed
ascend-robot
4月8日 评论：
4月8日 评论：
流水线 PR-pipeline_pytorch#14633 已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| Build_ARM_inductor | ✅ | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | ✅ | >>> |
| UT_X86_Part_02 | ✅ | >>> | |
| UT_ARM_A3_Part_01 | ✅ | >>> | |
| UT_ARM_A3_Part_02 | ✅ | >>> | |
| UT_DIST_X86_Part_01 | 🛑 | >>> | |
| UT_DIST_X86_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_03 | 🛑 | >>> | |
| UT_DIST_X86_Part_04 | 🛑 | >>> | |
| UT_inductor_Part_01 | ✅ | >>> | |
| UT_inductor_Part_02 | ✅ | >>> | |
| UT_inductor_Part_03 | ✅ | >>> | |
| UT_inductor_Part_04 | ✅ | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
rain-666
4月8日 评论：
4月8日 评论：
/lgtm
4月8日 添加了label：lgtm
dezheng889
4月8日 评论：
4月8日 评论：
/approve
4月8日 添加了label：approved
ascend-robot
4月8日 评论：
4月8日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: crazyDannyBoy.
Reviewers who wrote a comment of /lgtm are: crazyDannyBoy, weizhan4, rain-666.
4月8日 合入了pull request