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
[sym_shapes][perf] Use sympy xreplace instead of subs #124208
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/124208
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 013ad44 with merge base 72271fb (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ghstack-source-id: 28268c69a15cab505f9927efdd531c25ee37ac05 Pull Request resolved: #124208
@IvanKobzarev has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
cc @asmeurer |
looks like you need to make the xreplace dicts |
sympy/sympy#22240 Differential Revision: [D56207553](https://our.internmc.facebook.com/intern/diff/D56207553) [ghstack-poisoned]
ghstack-source-id: 8a282cc426a0ed5db0ebceff1a51e229bac55eb2 Pull Request resolved: #124208
@IvanKobzarev has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
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.
Would it be too mad to write a regression test that looks at the source of symbolic_shapes.py
and perhaps the files under the utils/_sympy
folder and makes sure we are not using sub
?
This is a dupe of #124207. Which one are we going with? |
@pytorchbot merge -f 'Landed internally' (Initiating merge automatically since Phabricator Diff has merged, using force because this PR might not pass merge_rules.json but landed internally) |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
sympy/sympy#22240 Differential Revision: [D56207553](https://our.internmc.facebook.com/intern/diff/D56207553) Pull Request resolved: #124208 Approved by: https://github.com/ezyang, https://github.com/lezcano
sympy/sympy#22240 Differential Revision: [D56207553](https://our.internmc.facebook.com/intern/diff/D56207553) Pull Request resolved: #124208 Approved by: https://github.com/ezyang, https://github.com/lezcano
sympy/sympy#22240 Differential Revision: [D56207553](https://our.internmc.facebook.com/intern/diff/D56207553) Pull Request resolved: pytorch#124208 Approved by: https://github.com/ezyang, https://github.com/lezcano
sympy/sympy#22240 Differential Revision: [D56207553](https://our.internmc.facebook.com/intern/diff/D56207553) Pull Request resolved: #124208 Approved by: https://github.com/ezyang, https://github.com/lezcano
Stack from ghstack (oldest at bottom):
sympy/sympy#22240
Differential Revision: D56207553