-
Notifications
You must be signed in to change notification settings - Fork 212
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
feat: Update NCCL to v2.21.5
#1780
Conversation
Please help integrate #1823's ARM related changes as well, thanks! |
@nWEIdia done! |
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.
Looks great!
Once this is merged, feel free to merge my PR on PyTorch |
Could you please do another rebase? @bryantbiggs @Skylion007 we may need to repeat a round of pytorch/pytorch test after Bryant rebases. |
@nWEIdia done! |
b8a0904
to
7e46db4
Compare
rebased to include CI fixes from #1885 |
looks like the failed test is a flake if someone could re-run the failed tests
|
Supporting pytorch/pytorch#124014
Changelog https://docs.nvidia.com/deeplearning/nccl/release-notes/rel_2-21-5.html#rel_2-21-5