make to_edge_transform_and_lower support etrecord generation#13034
Conversation
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/13034
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 2 New FailuresAs of commit 075341a with merge base 8651d31 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) ghstack-source-id: 299779126 Pull Request resolved: #13034
This PR needs a
|
|
This pull request was exported from Phabricator. Differential Revision: D79336982 |
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) umbrella issue: #12961 [ghstack-poisoned]
Pull Request resolved: #13034 This diff focus on making `to_edge_transform_and_lower` pipeline support etrecord generation, which can not work on etrecord generation at all previously. Also add tests into it. ghstack-source-id: 299899083 @exported-using-ghexport Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/)
|
This pull request was exported from Phabricator. Differential Revision: D79336982 |
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) umbrella issue: #12961 [ghstack-poisoned]
Pull Request resolved: #13034 This diff focus on making `to_edge_transform_and_lower` pipeline support etrecord generation, which can not work on etrecord generation at all previously. Also add tests into it. ghstack-source-id: 299903999 @exported-using-ghexport Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/)
|
This pull request was exported from Phabricator. Differential Revision: D79336982 |
|
It might be worth adding a test that covers to_edge_transform_and_lower with a partitioner. The logic might already handle this case, but to_backend creates a new edge program manager and is only called internally when a partitioner is provided. It might need some extra logic for this case to propagate the edge manager in to_backend. |
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) umbrella issue: #12961 [ghstack-poisoned]
|
This pull request was exported from Phabricator. Differential Revision: D79336982 |
Differential Revision: [D79336982](https://our.internmc.facebook.com/intern/diff/D79336982/) umbrella issue: #12961 [ghstack-poisoned]
|
This pull request was exported from Phabricator. Differential Revision: D79336982 |
cabfef4
into
gh/gasoonjia/32/base
Stack from ghstack (oldest at bottom):
Differential Revision: D79336982
umbrella issue: #12961