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

Forward-merge branch-24.10 into branch-24.12 #351

Merged
merged 1 commit into from
Sep 25, 2024
Merged

Commits on Sep 25, 2024

  1. remove NCCL pins in build and test environments (#341)

    Contributes to rapidsai/build-planning#102
    
    Some RAPIDS libraries are using `ncclCommSplit()`, which was introduced in `nccl==2.18.1.1`. This is part of a series of PRs across RAPIDS updating libraries' pins to `nccl>=2.18.1.1` to ensure they get a new-enough version that supports that.
    
    `cuvs` doesn't have any *direct* uses of NCCL... it only uses it via raft. This PR proposes removing `cuvs`'s dependency pinnings on NCCL, in favor of just using whatever it gets transitively via raft.
    
    Authors:
      - James Lamb (https://github.com/jameslamb)
    
    Approvers:
      - Vyas Ramasubramani (https://github.com/vyasr)
      - Corey J. Nolet (https://github.com/cjnolet)
    
    URL: #341
    jameslamb authored Sep 25, 2024
    Configuration menu
    Copy the full SHA
    8e02150 View commit details
    Browse the repository at this point in the history