-
Notifications
You must be signed in to change notification settings - Fork 403
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
Fix torch._inductor.aoti_compile_and_package input #7400
base: main
Are you sure you want to change the base?
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/7400
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New FailuresAs of commit c14f801 with merge base f341da8 (): NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
0a16dde
to
e4a34eb
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
e4a34eb
to
9ca54c5
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
9ca54c5
to
38c983f
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
@@ -1 +1 @@ | |||
19eff28ff3f19b50da46f5a9ff5f4d4d213806fe | |||
288aa873831057b1eb7d747914ec4fdc76c23a80 |
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.
Should update this and the torch pin in a separate pr
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.
@dvorjackz but if I update the torch pin without other changes in the PR, the tests will fail due to the input to torch._inductor.aoti_compile_and_package changed?
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.
True actually, let's use this instead though actually #7273, seems better for the sake of updating the pin correctly
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
38c983f
to
f450d8d
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
f450d8d
to
8294a2f
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
Summary: The inputs to `torch._inductor.aoti_compile_and_package` changed in pytorch/pytorch#140991. The `args` and `kwargs` do not need to be inputed anymore. The API will get them from `exported_program.example_inputs` Differential Revision: D67436429
8294a2f
to
26e9053
Compare
26e9053
to
c14f801
Compare
This pull request was exported from Phabricator. Differential Revision: D67436429 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D67436429 |
Summary:
The inputs to
torch._inductor.aoti_compile_and_package
changed in pytorch/pytorch#140991.The
args
andkwargs
do not need to be inputed anymore. The API will get them fromexported_program.example_inputs
Differential Revision: D67436429