-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[Gradient Compression] Update _powerSGD_comm_hook_wrapper to only expose 2 most critical hyperparameters #55295
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
Conversation
…ose 2 most critical hyperparameters Update `_powerSGD_comm_hook_wrapper` to only expose 2 most critical hyperparameters, to make this API more clear to any future user (although the second hyperparameter `start_powerSGD_iter` is not in use yet). Differential Revision: [D27561734](https://our.internmc.facebook.com/intern/diff/D27561734/) [ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit 851661a (more details on the Dr. CI page):
🕵️ 4 new failures recognized by patternsThe following CI failures do not appear to be due to upstream breakages:
|
…ose 2 most critical hyperparameters Update `_powerSGD_comm_hook_wrapper` to only expose 2 most critical hyperparameters, to make this API more clear to any future user (although the second hyperparameter `start_powerSGD_iter` is not in use yet). Differential Revision: [D27561734](https://our.internmc.facebook.com/intern/diff/D27561734/) ghstack-source-id: 125707743 Pull Request resolved: #55295
This pull request has been merged in 1b4bb36. |
Stack from ghstack:
Update
_powerSGD_comm_hook_wrapper
to only expose 2 most critical hyperparameters, to make this API more clear to any future user (although the second hyperparameterstart_powerSGD_iter
is not in use yet).Facebook: PyTorch STL/Lightning team once tried to use this API.
Differential Revision: D27561734