[aotinductor] Solves a problem where a tensor is returned more than once #112177

desertfire · 2023-10-26T17:51:47Z

Stack from ghstack (oldest at bottom):

-> [aotinductor] Solves a problem where a tensor is returned more than once #112177

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @peterbell10 @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @aakhundov @ColinPeppler @avikchaudhuri @gmagogsfm @zhxchen17 @tugsbayasgalan @angelayi @suo @ydwu4

[ghstack-poisoned]

pytorch-bot · 2023-10-26T17:51:50Z

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/112177

📄 Preview Python docs built from this PR
📄 Preview C++ docs built from this PR
❓ Need help or want to give feedback on the CI? Visit the bot commands wiki or our office hours

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (5 Unrelated Failures)

As of commit 065fe82 with merge base 7f143d7 ():

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

…more than once" cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 aakhundov ColinPeppler avikchaudhuri gmagogsfm zhxchen17 tugsbayasgalan angelayi [ghstack-poisoned]

ghstack-source-id: e120a4e Pull Request resolved: #112177

desertfire · 2023-10-31T13:55:31Z

@pytorchbot merge

pytorchmergebot · 2023-10-31T13:57:53Z

Merge failed

Reason: This PR needs a release notes: label
If your changes are user facing and intended to be a part of release notes, please use a label starting with release notes:.

If not, please add the topic: not user facing label.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "topic: not user facing"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Details for Dev Infra team

Raised by workflow job

desertfire · 2023-11-01T13:05:47Z

@pytorchbot merge

pytorchmergebot · 2023-11-01T13:07:38Z

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

pytorchmergebot · 2023-11-01T13:08:09Z

Merge failed

Reason: Command git -C /home/runner/work/pytorch/pytorch cherry-pick -x 771435ac9cc0676c6446f831abe61af59a09357c returned non-zero exit code 1

Auto-merging test/inductor/test_aot_inductor.py
CONFLICT (content): Merge conflict in test/inductor/test_aot_inductor.py
error: could not apply 771435ac9cc... [aotinductor] Solves a problem where a tensor is returned more than once
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

Details for Dev Infra team

Raised by workflow job

…more than once" cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 aakhundov ColinPeppler avikchaudhuri gmagogsfm zhxchen17 tugsbayasgalan angelayi suo [ghstack-poisoned]

ghstack-source-id: de9d463 Pull Request resolved: #112177

…more than once" cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx peterbell10 ipiszy yf225 chenyang78 kadeng muchulee8 aakhundov ColinPeppler avikchaudhuri gmagogsfm zhxchen17 tugsbayasgalan angelayi suo [ghstack-poisoned]

desertfire · 2023-11-03T02:28:45Z

@pytorchbot merge

desertfire · 2023-11-03T18:23:17Z

@pytorchbot merge

pytorchmergebot · 2023-11-03T18:25:36Z

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

PaliC · 2023-11-06T06:16:51Z

@pytorchbot revert -m "breaking internal tests (refer to internal diff)" -c "ghfirst"

pytorchmergebot · 2023-11-06T06:20:27Z

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

pytorchmergebot · 2023-11-06T06:20:38Z

@desertfire your PR has been successfully reverted.

…e than once (#112177)" This reverts commit a91baaf. Reverted #112177 on behalf of https://github.com/PaliC due to breaking internal tests (refer to internal diff) ([comment](#112177 (comment)))

desertfire · 2023-11-06T18:18:02Z

@pytorchbot rebase

pytorchmergebot · 2023-11-06T18:19:41Z

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

pytorchmergebot · 2023-11-06T18:19:45Z

Rebase failed due to

Aborting rebase because rebasing the branch resulted in the same sha as the target branch.
This usually happens because the PR has already been merged.  Please rebase locally and push.

Raised by https://github.com/pytorch/pytorch/actions/runs/6774873717

desertfire · 2023-11-06T19:03:34Z

The internal test failure is flaky and doesn't seem to be related. @pytorchbot merge

desertfire · 2023-11-06T20:09:48Z

@pytorchbot merge

pytorchmergebot · 2023-11-06T20:11:40Z

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

…e than once (pytorch#112177)" This reverts commit a91baaf. Reverted pytorch#112177 on behalf of https://github.com/PaliC due to breaking internal tests (refer to internal diff) ([comment](pytorch#112177 (comment)))

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

…e than once (pytorch#112177)" This reverts commit a91baaf. Reverted pytorch#112177 on behalf of https://github.com/PaliC due to breaking internal tests (refer to internal diff) ([comment](pytorch#112177 (comment)))

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

[aotinductor] Solves a problem where a tensor is returned more than once

286242f

[ghstack-poisoned]

desertfire requested review from avikchaudhuri, gmagogsfm, tugsbayasgalan and zhxchen17 as code owners October 26, 2023 17:51

github-actions bot added module: inductor ciflow/inductor module: export labels Oct 26, 2023

desertfire requested a review from angelayi October 26, 2023 18:32

desertfire added a commit that referenced this pull request Oct 26, 2023

[aotinductor] Solves a problem where a tensor is returned more than once

771435a

ghstack-source-id: e120a4e Pull Request resolved: #112177

zhxchen17 approved these changes Oct 30, 2023

View reviewed changes

pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 31, 2023

pytorchmergebot added the merging label Oct 31, 2023

pytorchmergebot removed the merging label Oct 31, 2023

desertfire added the topic: not user facing topic category label Oct 31, 2023

pytorchmergebot added the merging label Nov 1, 2023

pytorchmergebot removed the merging label Nov 1, 2023

desertfire added a commit that referenced this pull request Nov 1, 2023

[aotinductor] Solves a problem where a tensor is returned more than once

f98a680

ghstack-source-id: de9d463 Pull Request resolved: #112177

pytorchmergebot added the merging label Nov 3, 2023

pytorchmergebot added Merged and removed merging labels Nov 3, 2023

pytorchmergebot closed this in a91baaf Nov 3, 2023

pytorchmergebot added the Reverted label Nov 6, 2023

pytorchmergebot reopened this Nov 6, 2023

pytorchmergebot added the merging label Nov 6, 2023

pytorchmergebot removed the merging label Nov 6, 2023

pytorchmergebot closed this in 67256d5 Nov 6, 2023

xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Nov 7, 2023

[aotinductor] Solves a problem where a tensor is returned more than o…

965eb50

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Nov 7, 2023

[aotinductor] Solves a problem where a tensor is returned more than o…

1172421

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

facebook-github-bot deleted the gh/desertfire/249/head branch November 10, 2023 15:24

Skylion007 pushed a commit to Skylion007/pytorch that referenced this pull request Nov 14, 2023

[aotinductor] Solves a problem where a tensor is returned more than o…

a0ea62e

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

Skylion007 pushed a commit to Skylion007/pytorch that referenced this pull request Nov 14, 2023

[aotinductor] Solves a problem where a tensor is returned more than o…

20177c5

…nce (pytorch#112177) Pull Request resolved: pytorch#112177 Approved by: https://github.com/zhxchen17

[aotinductor] Solves a problem where a tensor is returned more than once #112177

[aotinductor] Solves a problem where a tensor is returned more than once #112177

Uh oh!

Conversation

desertfire commented Oct 26, 2023 • edited by pytorch-bot bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pytorch-bot bot commented Oct 26, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/112177

✅ You can merge normally! (5 Unrelated Failures)

Uh oh!

desertfire commented Oct 31, 2023

Uh oh!

pytorchmergebot commented Oct 31, 2023

Merge failed

Uh oh!

desertfire commented Nov 1, 2023

Uh oh!

pytorchmergebot commented Nov 1, 2023

Merge started

Uh oh!

pytorchmergebot commented Nov 1, 2023

Merge failed

Uh oh!

desertfire commented Nov 3, 2023

Uh oh!

desertfire commented Nov 3, 2023

Uh oh!

pytorchmergebot commented Nov 3, 2023

Merge started

Uh oh!

PaliC commented Nov 6, 2023

Uh oh!

pytorchmergebot commented Nov 6, 2023

Uh oh!

pytorchmergebot commented Nov 6, 2023

Uh oh!

desertfire commented Nov 6, 2023

Uh oh!

pytorchmergebot commented Nov 6, 2023

Uh oh!

pytorchmergebot commented Nov 6, 2023

Uh oh!

desertfire commented Nov 6, 2023

Uh oh!

desertfire commented Nov 6, 2023

Uh oh!

pytorchmergebot commented Nov 6, 2023

Merge started

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

desertfire commented Oct 26, 2023 •

edited by pytorch-bot bot

Loading

pytorch-bot bot commented Oct 26, 2023 •

edited

Loading