New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed minor issues for bmm/mm decompositon #109836
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/109836
Note: Links to docs will display an error until the docs builds have been completed. ⏳ No Failures, 1 PendingAs of commit 0801683 with merge base 7f5fd92 (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D49522332 |
torch/_inductor/decomposition.py
Outdated
@@ -29,6 +29,7 @@ | |||
inductor_decompositions = get_decompositions( | |||
[ | |||
aten._adaptive_avg_pool2d_backward, | |||
aten.addmm, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is causing the test failures.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, quite a few accuracy issues. I remove this part from this PR. It may take more time to triage and fix those issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In #110010 I just added a custom addmm decomp for this case. I'm not sure what changed since July that this is needed.
82b3416
to
3221ce7
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
3221ce7
to
7111038
Compare
This pull request was exported from Phabricator. Differential Revision: D49522332 |
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
f50ed28
to
b39f66f
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D49522332 |
b39f66f
to
aa49473
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
aa49473
to
865c980
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D49522332 |
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
865c980
to
66e3f17
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
51b1f61
to
950ea7a
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
950ea7a
to
bde7066
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D49522332 |
@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: 14 mandatory check(s) failed. The first few are:
Dig deeper by viewing the failures on hud |
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
bde7066
to
c0c14b0
Compare
Summary: * Fixed minor issues for bmm/mm decompositon * enabled addmm for inductor Test Plan: ci Reviewed By: mikekgfb Differential Revision: D49522332
This pull request was exported from Phabricator. Differential Revision: D49522332 |
c0c14b0
to
0801683
Compare
This pull request was exported from Phabricator. Differential Revision: D49522332 |
@pytorchbot merge -f 'Landed internally' (Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally) |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Summary:
Test Plan: ci
Reviewed By: mikekgfb
Differential Revision: D49522332
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @Xia-Weiwen @wenzhe-nrv @jiayisunx @peterbell10 @ipiszy @yf225 @kadeng @muchulee8 @aakhundov @ColinPeppler @ngimel