-
Notifications
You must be signed in to change notification settings - Fork 22.1k
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
[JIT] Add support for backend-lowered submodules #41146
Conversation
**Summary** This commit adds support for using `Modules` that have been lowered as submodules in `ScriptModules`. **Test Plan** This commit adds execution and save/load tests to test_backends.py for backend-lowered submodules. **Fixes** This commit fixes #40069. [ghstack-poisoned]
**Summary** This commit adds support for using `Modules` that have been lowered as submodules in `ScriptModules`. **Test Plan** This commit adds execution and save/load tests to test_backends.py for backend-lowered submodules. **Fixes** This commit fixes #40069. ghstack-source-id: b6aabcfedb34e1f68b26b0c8a546298d5c41152c Pull Request resolved: #41146
@suo This is the one that got reverted but somehow the tests are clean now without any extra changes? I don't know what's going on :( |
💊 CI failures summary and remediationsAs of commit 262ee36 (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 12 times. |
**Summary** This commit adds support for using `Modules` that have been lowered as submodules in `ScriptModules`. **Test Plan** This commit adds execution and save/load tests to test_backends.py for backend-lowered submodules. **Fixes** This commit fixes #40069. Differential Revision: [D22459543](https://our.internmc.facebook.com/intern/diff/D22459543) [ghstack-poisoned]
**Summary** This commit adds support for using `Modules` that have been lowered as submodules in `ScriptModules`. **Test Plan** This commit adds execution and save/load tests to test_backends.py for backend-lowered submodules. **Fixes** This commit fixes #40069. Differential Revision: [D22459543](https://our.internmc.facebook.com/intern/diff/D22459543) [ghstack-poisoned]
**Summary** This commit adds support for using `Modules` that have been lowered as submodules in `ScriptModules`. **Test Plan** This commit adds execution and save/load tests to test_backends.py for backend-lowered submodules. **Fixes** This commit fixes #40069. ghstack-source-id: 8b8ad87d29b0ca31e9d8e79b2012fe51eb0f78e7 Pull Request resolved: #41146
@SplitInfinity merged this pull request in ce3ba3b. |
Stack from ghstack:
Summary
This commit adds support for using
Modules
that have been lowered assubmodules in
ScriptModules
.Test Plan
This commit adds execution and save/load tests to test_backends.py for
backend-lowered submodules.
Fixes
This commit fixes #40069.
Differential Revision: D22459543