Skip to content

Upgrade NCCL to 2.9.9 - #3902

Merged
rapids-bot[bot] merged 7 commits into
NVIDIA:branch-21.06from
dantegd:2106-nccl299
Jun 4, 2021
Merged

Upgrade NCCL to 2.9.9 #3902
rapids-bot[bot] merged 7 commits into
NVIDIA:branch-21.06from
dantegd:2106-nccl299

Conversation

@dantegd

@dantegd dantegd commented May 26, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@dantegd dantegd added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 26, 2021
@dantegd
dantegd requested a review from a team as a code owner May 26, 2021 15:28
@github-actions github-actions Bot added conda conda issue gpuCI gpuCI issue labels May 26, 2021
@cjnolet

cjnolet commented May 26, 2021

Copy link
Copy Markdown
Collaborator

Looks like the CI failures here are related to recent Dask changes, which were recently fixed here?

Going to rerun tests to see.

@cjnolet

cjnolet commented May 26, 2021

Copy link
Copy Markdown
Collaborator

rerun tests

1 similar comment
@dantegd

dantegd commented May 27, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@dantegd

dantegd commented May 28, 2021

Copy link
Copy Markdown
Contributor Author

rerun tests

@jakirkham jakirkham left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Swapping which one is dev installed

Comment thread ci/gpu/build.sh
Comment thread ci/gpu/build.sh Outdated
@dantegd dantegd changed the title Testing upgrade NCCL to 2.9.9 Testing upgrade NCCL to 2.9.9 [skip-ci] May 28, 2021
Co-authored-by: jakirkham <jakirkham@gmail.com>
@dantegd dantegd changed the title Testing upgrade NCCL to 2.9.9 [skip-ci] Testing upgrade NCCL to 2.9.9 May 28, 2021
Co-authored-by: jakirkham <jakirkham@gmail.com>
Comment thread ci/gpu/build.sh Outdated
gpuci_logger "Install the main version of dask and distributed"
set -x
pip install "git+https://github.com/dask/distributed.git@main" --upgrade --no-deps
pip install "git+https://github.com/jakirkham/distributed.git@drop_whowhat_client" --upgrade --no-deps

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be reverted once ready as it is merged

Suggested change
pip install "git+https://github.com/jakirkham/distributed.git@drop_whowhat_client" --upgrade --no-deps
pip install "git+https://github.com/dask/distributed.git@main" --upgrade --no-deps

@dantegd dantegd changed the title Testing upgrade NCCL to 2.9.9 Upgrading NCCL to 2.9.9 May 28, 2021
@dantegd dantegd changed the title Upgrading NCCL to 2.9.9 Upgrade NCCL to 2.9.9 May 28, 2021
@dantegd

dantegd commented May 29, 2021

Copy link
Copy Markdown
Contributor Author

Note: PR is waiting for an equivalent in integration

@github-actions github-actions Bot removed the gpuCI gpuCI issue label May 31, 2021
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.06@95efa25). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.06    #3902   +/-   ##
===============================================
  Coverage                ?   85.07%           
===============================================
  Files                   ?      228           
  Lines                   ?    17775           
  Branches                ?        0           
===============================================
  Hits                    ?    15122           
  Misses                  ?     2653           
  Partials                ?        0           
Flag Coverage Δ
dask 48.18% <0.00%> (?)
non-dask 77.26% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95efa25...be75d19. Read the comment docs.

@cjnolet

cjnolet commented Jun 1, 2021

Copy link
Copy Markdown
Collaborator

This change has been approved, is it good to go for the release?

@dantegd

dantegd commented Jun 1, 2021

Copy link
Copy Markdown
Contributor Author

It's just waiting on rapidsai/integration#287 so that CI doesn't get issues after the merge

@dantegd

dantegd commented Jun 4, 2021

Copy link
Copy Markdown
Contributor Author

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit c27f132 into NVIDIA:branch-21.06 Jun 4, 2021
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conda conda issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants