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
[torch/elastic] Revise the rendezvous handler registry logic. #55466
Conversation
💊 CI failures summary and remediationsAs of commit 14d1f84 (more details on the Dr. CI page):
ci.pytorch.org: 1 failedThis 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 pull request was exported from Phabricator. Differential Revision: D27623215 |
…h#55466) Summary: Pull Request resolved: pytorch#55466 Improve the implementation and the unit test coverage of `RendezvousHandlerRegistry`. ### Note See the original diff (D27442325 (pytorch@df299db)) that had to be reverted due to an unexpected Python version incompatibility between the internal and external PyTorch CI tests. Test Plan: Run the existing and newly-introduced unit tests. Differential Revision: D27623215 fbshipit-source-id: fd31db5373793396124df02f83c1d4c420142f04
c362aab
to
14d1f84
Compare
This pull request was exported from Phabricator. Differential Revision: D27623215 |
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.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #55466 +/- ##
==========================================
- Coverage 77.41% 76.99% -0.43%
==========================================
Files 1895 1895
Lines 187583 187593 +10
==========================================
- Hits 145215 144433 -782
- Misses 42368 43160 +792 |
This pull request has been merged in 493a233. |
Summary:
Improve the implementation and the unit test coverage of
RendezvousHandlerRegistry
.Note
See the original diff (D27442325 (df299db)) that had to be reverted due to an unexpected Python version incompatibility between the internal and external PyTorch CI tests.
Test Plan: Run the existing and newly-introduced unit tests.
Differential Revision: D27623215