-
Notifications
You must be signed in to change notification settings - Fork 21.6k
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
Migrate some torch.fft tests to use OpInfos #48428
Conversation
[ghstack-poisoned]
[ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit dbb909a (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions to the (internal) Dr. CI Users group. This comment has been revised 82 times. |
[ghstack-poisoned]
ghstack-source-id: 21d8415bf219382a5100b5a4907ad1409e0ded2c Pull Request resolved: #48428
[ghstack-poisoned]
[ghstack-poisoned]
ghstack-source-id: 11a427ea77e5fb307fd2f6dcfc7b82051accbcef Pull Request resolved: #48428
[ghstack-poisoned]
ghstack-source-id: 054cd4d89887575c79800f4839e9a6f41c7bf500 Pull Request resolved: #48428
[ghstack-poisoned]
[ghstack-poisoned]
ghstack-source-id: 3b9f7dd58817e35c52945fb3fdf14c8ceb7eb4a3 Pull Request resolved: #48428
[ghstack-poisoned]
ghstack-source-id: 3543e29ebb4d41067b9dd371e62e8aaa76382156 Pull Request resolved: #48428
Hey @peterbell10! This is really cool. I have a question about how we want to test backward, however, that I'd like to hear your thoughts on. Ideally autograd (and soon the jit) will only need testing in test_ops.py. |
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
ghstack-source-id: 172f598a12945ff76ec891541fa2c0433dad6a56 Pull Request resolved: pytorch#48428
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
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.
Cool!
Stack from ghstack:
Differential Revision: D25868666