[inductor] Fix "RuntimeError: Tried to erase Node permute but it still had 3 users in the graph" #91327
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/91327
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 FailuresAs of commit 0ac5a31: FLAKY - The following jobs failed but were likely due to flakiness present on master:
BROKEN TRUNK - The following jobs failed but were present on the merge base ca39c5b:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This pull request was exported from Phabricator. Differential Revision: D42213859 |
|
Can you please add a testcase based on @anijain2305 minifier? You can patch |
jiawenliu64
force-pushed
the
export-D42213859
branch
from
4dd1ad7 to
ca6629a
Compare
|
This pull request was exported from Phabricator. Differential Revision: D42213859 |
|
Synced with @anijain2305: the issue is addressed with tests in minifier, and add tests based on minifier |
…l had 3 users in the graph" (pytorch#91327) Summary: Pull Request resolved: pytorch#91327 Fix "RuntimeError: Tried to erase Node permute but it still had 3 users in the graph" to unblock internal models Reviewed By: anijain2305, ngimel Differential Revision: D42213859 fbshipit-source-id: ceba64b40a91c37c84e693d640a9e210a9db2365
jiawenliu64
force-pushed
the
export-D42213859
branch
from
ca6629a to
73e081f
Compare
|
This pull request was exported from Phabricator. Differential Revision: D42213859 |
|
It seems all test failures are due to the sev (#91332). Will check when the sev is solved |
…l had 3 users in the graph" (pytorch#91327) Summary: Pull Request resolved: pytorch#91327 Fix "RuntimeError: Tried to erase Node permute but it still had 3 users in the graph" to unblock internal models Reviewed By: anijain2305, jianyuh, ngimel Differential Revision: D42213859 fbshipit-source-id: ae02b2c75cf67e275fb5dd841a238847904a53d9
jiawenliu64
force-pushed
the
export-D42213859
branch
from
73e081f to
0ac5a31
Compare
|
This pull request was exported from Phabricator. Differential Revision: D42213859 |
|
@pytorchbot merge |
Merge startedYour 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 |
Merge failedReason: The following mandatory check(s) failed (Rule Dig deeper by viewing the failures on hud Details for Dev Infra teamRaised by workflow job |
|
@pytorchbot merge -f "failure unrelated" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
6 participants
Summary: Fix "RuntimeError: Tried to erase Node permute but it still had 3 users in the graph" to unblock internal models
Differential Revision: D42213859
cc @mlazos @soumith @voznesenskym @yanboliang @penguinwu @anijain2305 @EikanWang @jgong5 @Guobing-Chen @chunyuan-w @XiaobingSuper @zhuhaozhe @blzheng @Xia-Weiwen @wenzhe-nrv @jiayisunx @peterbell10 @desertfire