Update NCCL from 2.4.8 to 2.7.3 #41608
Conversation
There was a problem hiding this comment.
@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
💊 CI failures summary and remediationsAs of commit 9ae63a3 (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 on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 14 times. |
There was a problem hiding this comment.
There are test failures when bumping up NCCL version since 2.6.4.
@agolynski is working on fixing it. See #37506 and #40622
malfet
force-pushed
the
malfet/update-nccl
branch
from
fbd552e
to
56cffb9
Compare
There was a problem hiding this comment.
@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
malfet
force-pushed
the
malfet/update-nccl
branch
from
56cffb9
to
b59b046
Compare
There was a problem hiding this comment.
@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
malfet
force-pushed
the
malfet/update-nccl
branch
from
b59b046
to
9ae63a3
Compare
malfet
changed the title
There was a problem hiding this comment.
@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
@agolynski mentioned that NCCL 2.7 might drop support for cuda 9.2, but we still have cuda 9.2 tests on master. Do we need to worry about this? @malfet @ngimel |
No description provided.