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
[dynamo] Fix test #125107
[dynamo] Fix test #125107
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/125107
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New FailureAs of commit 24a77fe with merge base bad8d25 (): NEW FAILURE - The following job has failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
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.
What's wrong with this test?
Also, can we get #125062 in first? It adds an xfail to 3.12 and makes Mod.forward
refer to self.fc
instead of fc
.
The test was returning input as well as fc. And we were passing both the input and the fc to the module call. This was exposed in a PR in this stack. I am fine rebasing after your PR goes in. |
Earlier globals of inlined functions from other files were not handled correctly. We were not tracking mutations on them. They were colliding with the same global name in the parent function etc. This PR overrides the LOAD/STORE_GLOBAL for inline tx and tracks mutation on them separately. Pull Request resolved: #125002 Approved by: https://github.com/jansel ghstack dependencies: #125097, #125107
Pull Request resolved: pytorch#125107 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097
Earlier globals of inlined functions from other files were not handled correctly. We were not tracking mutations on them. They were colliding with the same global name in the parent function etc. This PR overrides the LOAD/STORE_GLOBAL for inline tx and tracks mutation on them separately. Pull Request resolved: pytorch#125002 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097, pytorch#125107
Pull Request resolved: pytorch#125107 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097
Earlier globals of inlined functions from other files were not handled correctly. We were not tracking mutations on them. They were colliding with the same global name in the parent function etc. This PR overrides the LOAD/STORE_GLOBAL for inline tx and tracks mutation on them separately. Pull Request resolved: pytorch#125002 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097, pytorch#125107
Pull Request resolved: pytorch#125107 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097
Earlier globals of inlined functions from other files were not handled correctly. We were not tracking mutations on them. They were colliding with the same global name in the parent function etc. This PR overrides the LOAD/STORE_GLOBAL for inline tx and tracks mutation on them separately. Pull Request resolved: pytorch#125002 Approved by: https://github.com/jansel ghstack dependencies: pytorch#125097, pytorch#125107
Stack from ghstack (oldest at bottom):
cc @ezyang @msaroufim @bdhirsh @chauhang @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng