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
[Gradient Compression] Add CppCommHook subclass for supporting the C++ API of communication hook. #46566
Conversation
…+ API of communication hook. Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit 45084ff (more details on the Dr. CI page): Commit 45084ff was recently pushed. Waiting for builds... 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 on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 1 time. |
💊 CI failures summary and remediationsAs of commit c78c03a (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 1 failure confirmed as flaky and can be ignored:
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 on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 46 times. |
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…+ API of communication hook. Pull Request resolved: #46566 Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 ghstack-source-id: 114668384 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/)
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…+ API of communication hook. Pull Request resolved: #46566 Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 ghstack-source-id: 114687935 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/)
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…+ API of communication hook. Pull Request resolved: #46566 Only provides an interface. Some built-in implementations will be provided in a follow-up commit. Original PR issue: C++ DDP Communication Hook #46348 ghstack-source-id: 115213919 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/)
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
…ting the C++ API of communication hook." Only provides an interface. Some built-in implementations will be provided in a follow-up commit. riginal PR issue: C++ DDP Communication Hook #46348 Differential Revision: [D24379460](https://our.internmc.facebook.com/intern/diff/D24379460/) [ghstack-poisoned]
This pull request has been merged in e077a2a. |
Stack from ghstack:
Only provides an interface. Some built-in implementations will be provided in a follow-up commit.
riginal PR issue: C++ DDP Communication Hook #46348
Differential Revision: D24379460