-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[export] Move swap to a different file #137134
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
Conversation
Refactor so that unflattener doesn't become too messy Differential Revision: [D63719648](https://our.internmc.facebook.com/intern/diff/D63719648/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/137134
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 01d982b with merge base f2d174c ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Refactor so that unflattener doesn't become too messy Differential Revision: [D63719648](https://our.internmc.facebook.com/intern/diff/D63719648/) ghstack-source-id: 245757116 Pull Request resolved: #137134
|
This pull request was exported from Phabricator. Differential Revision: D63719648 |
avikchaudhuri
left a comment
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 the status of being able to import things from torch.export.unflatten? Since it is both a module as well as a function...
I have no clue.. I don't really know how that works.. |
Refactor so that unflattener doesn't become too messy Differential Revision: [D63719648](https://our.internmc.facebook.com/intern/diff/D63719648/) [ghstack-poisoned]
|
This pull request was exported from Phabricator. Differential Revision: D63719648 |
Refactor so that unflattener doesn't become too messy Differential Revision: [D63719648](https://our.internmc.facebook.com/intern/diff/D63719648/) [ghstack-poisoned]
Pull Request resolved: #137134 Refactor so that unflattener doesn't become too messy ghstack-source-id: 246432900 Differential Revision: [D63719648](https://our.internmc.facebook.com/intern/diff/D63719648/)
|
This pull request was exported from Phabricator. Differential Revision: D63719648 |
|
@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 |
Stack from ghstack (oldest at bottom):
Refactor so that unflattener doesn't become too messy
Differential Revision: D63719648