Merged
fix: 修复含子图节点(PartitionedCall)的悬空输出offset未正确回填的问题 #3661
yelongjianAdd on Jun 23
fix: 修复含子图节点(PartitionedCall)的悬空输出offset未正确回填的问题 #3661
Merged
yelongjianAdd on Jun 23
yelongjianCollaborators
Jun 23

Pull Request

描述

  1. 增加了PartitionedCallSuspendOut_WithPhonyConcatSubgraph用例,验证了原场景下会由于partitioned_call悬垂size和子图中的第二输出size不一致而报错
  2. 修改原来IDENTITY和READVARIABLEOP才回填offset的逻辑,改为包含子图的节点,获取到netoutput后对节点进行offset更新(含有子图的前提下)

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

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

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for yelongjian's contribution.)
YyelongjianCollaborators
Jun 23 Added label: enhancement
YyelongjianCollaborators
Jun 23 Created pull request, commit eda12d47
CANN-robotCANN-robotCollaborators
Jun 23 Added label: stat/needs-squash
CANN-robotCANN-robotCollaborators
Jun 23 Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
Jun 23 Comments:

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
repo-cann/ge ✅ wangxiaotian995, stevenaw0, tangqunzhang (3/2) ✅ wangxiaotian995 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
146 messages are collapsed here。 See more
atomgit-bot
atomgit-bot
Jun 23 Comments:

🤖 AI Code Review

⚠️ 审查未能完成

本次代码审查执行失败,未能生成结果。

可能原因:

  • 请求处理超时
  • 网络连接异常

建议:
请稍后重试。如果问题持续存在,请联系管理员。

追踪 ID: task-13414-run-12473

likedislike
Incorrect?
TtangqunzhangCollaborators
Jun 24 Resolved final issue
wangxiaotian995Collaborators
Jun 24 Comments:

/approve

likedislike
CANN-robotCANN-robotCollaborators
Jun 24 Added label: approved
CANN-robotCANN-robotCollaborators
Jun 24 Merged pull request