-
Notifications
You must be signed in to change notification settings - Fork 21.7k
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]: TSConverter errors on dynamic shapes #129087
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/129087
Note: Links to docs will display an error until the docs builds have been completed. ❌ 11 New FailuresAs of commit aac935d with merge base 67e22d6 (): NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
950a032
to
3d19c8a
Compare
fa8808a
to
41ab6fc
Compare
b65aef2
to
6c821a9
Compare
6c821a9
to
78fc411
Compare
torch/_subclasses/meta_utils.py
Outdated
@@ -1448,7 +1455,7 @@ def is_c_of_r(complex_dtype, real_dtype): | |||
device="meta", | |||
) | |||
) | |||
if self.copy_data: | |||
if self.copy_data and not has_symbolic_size(t.size): |
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.
Is @pianpwk the right person to confirm this is valid?
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.
definitely not lol
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.
could you make a separate PR with the changes in this file, and include a test case and have Ed take a look?
Description
Enable dynamic shape support during conversion.
meta_utils.py
for unbacked fake tensors.Test Plan
Add test cases that create dynamic shapes.
pytest test/export/test_converter.py -s -k test_ts2ep_dynamic_shape
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang @amjames