-
Notifications
You must be signed in to change notification settings - Fork 23.3k
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
[EdgeML] Remove dependency on all_mobile_model_configs.yaml from pt_operator_library BUCK rule #99122
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/99122
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: ✅ No FailuresAs of commit dff03d8: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D44946368 |
39e9520
to
25767d3
Compare
This pull request was exported from Phabricator. Differential Revision: D44946368 |
25767d3
to
5904570
Compare
This pull request was exported from Phabricator. Differential Revision: D44946368 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D44946368 |
5904570
to
3977e58
Compare
…perator_library BUCK rule (pytorch#99122) Summary: Pull Request resolved: pytorch#99122 Removes the dependency on the unified YAML file Test Plan: Smoke test via some caffe2 tests. ``` buck2 run xplat/caffe2:supported_mobile_models_test ``` Build a major FoA app that uses model tracing and confirm it still works. ``` buck2 build fb4a ``` CI/CD for the rest. If operator tracing / bundling was broken, I'd hope in the 1000+ tests spawned by this change should catch it. Reviewed By: dhruvbird Differential Revision: D44946368 fbshipit-source-id: bd1922fbf5ea38aa8ece2660477d901f5276deec
3977e58
to
dff03d8
Compare
This pull request was exported from Phabricator. Differential Revision: D44946368 |
@pytorchbot merge (Initiating merge automatically since Phabricator Diff has merged) |
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 |
Summary: Removes the dependency on the unified YAML file
Test Plan:
Smoke test via some caffe2 tests.
Build a major FoA app that uses model tracing and confirm it still works.
CI/CD for the rest. If operator tracing / bundling was broken, I'd hope in the 1000+ tests spawned by this change should catch it.
Differential Revision: D44946368