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
[optim/dynamo] shortcut adagrad with has_complex
#112722
[optim/dynamo] shortcut adagrad with has_complex
#112722
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/112722
Note: Links to docs will display an error until the docs builds have been completed. ⏳ No Failures, 1 PendingAs of commit 7fc0c32 with merge base 68dead4 (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
…uang/adagrad-shortcut-has-complex
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.
Thanks!
Any test needs updating?
There are existing tests for pytorch/test/optim/test_optim.py Line 1503 in a1ab22b
|
The compile + complex path is currently IIRC untested so that could honestly be improved. Created an issue: #112731 |
@pytorchbot merge |
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 |
Follow up to pytorch#110706, it was missed as depended on another fix Pull Request resolved: pytorch#112722 Approved by: https://github.com/albanD
Follow up to pytorch#110706, it was missed as depended on another fix Pull Request resolved: pytorch#112722 Approved by: https://github.com/albanD
Follow up to pytorch#110706, it was missed as depended on another fix Pull Request resolved: pytorch#112722 Approved by: https://github.com/albanD
Follow up to #110706, it was missed as depended on another fix
cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @aakhundov @kadeng