Skip to content
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

Support raft updating to new version of cuco #2360

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

ChuckHastings
Copy link
Collaborator

RAFT is updating to the newest version of cuco in this PR: rapidsai/raft#714

This PR makes the necessary changes to cugraph to support the latest cuco. It should be merged after the RAFT PR.

  • update version of cuco
  • update constructor calls to cuco::static_map

@ChuckHastings ChuckHastings requested review from a team as code owners June 16, 2022 20:19
@ChuckHastings ChuckHastings self-assigned this Jun 16, 2022
@ChuckHastings ChuckHastings added 3 - Ready for Review improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jun 16, 2022
@ChuckHastings ChuckHastings added this to the 22.08 milestone Jun 16, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 16, 2022

Codecov Report

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

@@               Coverage Diff               @@
##             branch-22.08    #2360   +/-   ##
===============================================
  Coverage                ?   60.76%           
===============================================
  Files                   ?      105           
  Lines                   ?     5077           
  Branches                ?        0           
===============================================
  Hits                    ?     3085           
  Misses                  ?     1992           
  Partials                ?        0           

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 9578cfe...6755661. Read the comment docs.

@cjnolet
Copy link
Member

cjnolet commented Jun 17, 2022

@ChuckHastings raft side has been merged so this should be good to go as well. I think this will also fix the current docker nightly dev container.

@BradReesWork
Copy link
Member

@gpucibot merge

@rapids-bot rapids-bot bot merged commit ee897c4 into rapidsai:branch-22.08 Jun 17, 2022
@ChuckHastings ChuckHastings deleted the try_latest_raft branch August 4, 2022 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants