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
[Docs] Minor doc fixes for init_process_group #47644
Conversation
Minor Update to the init_process_group docs. Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/) [ghstack-poisoned]
Minor Update to the init_process_group docs. Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/) ghstack-source-id: 116258381 Pull Request resolved: #47644
@@ -401,11 +401,11 @@ def init_process_group(backend, | |||
asynchronously and the process will crash. ``NCCL_BLOCKING_WAIT`` | |||
will provide errors to the user which can be caught and handled, | |||
but due to its blocking nature, it has a performance overhead. On | |||
the other hand, ``NCCL_ASYNC_ERROR_HANDLING`` has little | |||
the other hand, ``NCCL_ASYNC_ERROR_HANDLING`` has very little |
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.
redundant space before "very"
💊 CI failures summary and remediationsAs of commit 15549e7 (more details on the Dr. CI page):
🚧 2 fixed upstream failures:These were probably caused by upstream breakages that were already fixed.
Please rebase on the
|
Minor Update to the init_process_group docs. Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/) [ghstack-poisoned]
Pull Request resolved: #47644 Minor Update to the init_process_group docs. ghstack-source-id: 116337449 Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/)
Minor Update to the init_process_group docs. Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/) [ghstack-poisoned]
Pull Request resolved: #47644 Minor Update to the init_process_group docs. ghstack-source-id: 116441798 Differential Revision: [D24633432](https://our.internmc.facebook.com/intern/diff/D24633432/)
This pull request has been merged in 32b4b51. |
This pull request has been merged in 32b4b51. |
Stack from ghstack:
Minor Update to the init_process_group docs.
Differential Revision: D24633432