-
Notifications
You must be signed in to change notification settings - Fork 528
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
Revert D65620886 #3582
Revert D65620886 #3582
Conversation
This pull request was exported from Phabricator. Differential Revision: D68291310 |
✅ Deploy Preview for pytorch-fbgemm-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Summary: X-link: facebookresearch/FBGEMM#666 - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886 Reviewed By: leitian, jianyuh Differential Revision: D68291310
This pull request was exported from Phabricator. Differential Revision: D68291310 |
Summary: X-link: facebookresearch/FBGEMM#666 - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886 Reviewed By: leitian, jianyuh, yaoyj11 Differential Revision: D68291310
This pull request was exported from Phabricator. Differential Revision: D68291310 |
Summary: X-link: facebookresearch/FBGEMM#666 - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886 Reviewed By: leitian, jianyuh, yaoyj11 Differential Revision: D68291310
Summary: X-link: facebookresearch/FBGEMM#666 - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886 Reviewed By: leitian, jianyuh, yaoyj11 Differential Revision: D68291310
This pull request was exported from Phabricator. Differential Revision: D68291310 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D68291310 |
Summary: X-link: facebookresearch/FBGEMM#666 - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886 Reviewed By: leitian, jianyuh, yaoyj11 Differential Revision: D68291310
This pull request was exported from Phabricator. Differential Revision: D68291310 |
This pull request has been merged in 87db593. |
This reverts commit 87db593.
Summary: X-link: facebookresearch/FBGEMM#681 This PR: 1) Brings back the optimization reverted in #3582 2) Fixes the bug in tailing loop Pull Request resolved: #3599 Reviewed By: yaoyj11 Differential Revision: D68503146 Pulled By: q10 fbshipit-source-id: e7b35f109d72877272dee45b4c113b2bc2fbef8b
Summary: - Per discussions in https://fb.workplace.com/groups/fbgemmusers/permalink/9362556053825092/, revert D65620886
Reviewed By: jianyuh
Differential Revision: D68291310