Dependency refactor, add tests#1165
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/1165
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 9d52379 with merge base bfebeda ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
0122f64 to
adacaf1
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
adacaf1 to
a4a79fb
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
a4a79fb to
c9a257e
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
c9a257e to
37f89ba
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
37f89ba to
0b08637
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
0b08637 to
e1304cf
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
e1304cf to
d114133
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
d114133 to
392aa29
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
392aa29 to
9995feb
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
9995feb to
0005bbd
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
0005bbd to
6b497fa
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Differential Revision: D51036567
6b497fa to
a47a9f5
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Gated to only run with fbcode, as there is dependency on exir which is only in fbcode. Reviewed By: larryliu0820 Differential Revision: D51036567
a47a9f5 to
18cca74
Compare
Summary: Refactor dependency chain following: https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50 **New dep chain:** op_<op>.cpp -> selected_op_variants.h selected_op_variants.h -> scalar_type_util **Old dep chain:** op_<op>.cpp -> scalar_type_util scalar_type_util -> selected_op_variants.h **Testing** Add e2e test with addmul model Update unit test to use addmul model Gated to only run with fbcode, as there is dependency on exir which is only in fbcode. Reviewed By: larryliu0820 Differential Revision: D51036567
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
18cca74 to
9d52379
Compare
|
This pull request was exported from Phabricator. Differential Revision: D51036567 |
|
This pull request has been merged in f4679ae. |
Summary:
Refactor dependency chain following:
https://docs.google.com/document/d/1vuU9UdQA5bq3WU7YnEgJowq9ey6vkF3KUKA0rpIxxIY/edit#heading=h.8raqyft9y50
New dep chain:
op_.cpp -> selected_op_variants.h
selected_op_variants.h -> scalar_type_util
Old dep chain:
op_.cpp -> scalar_type_util
scalar_type_util -> selected_op_variants.h
Testing
Add e2e test with addmul model
Update unit test to use addmul model
Differential Revision: D51036567