Skip to content

Make cuVS optional if CUML_ALGORITHMS is set#6125

Merged
rapids-bot[bot] merged 3 commits intorapidsai:branch-24.12from
hcho3:optional_cuvs
Nov 4, 2024
Merged

Make cuVS optional if CUML_ALGORITHMS is set#6125
rapids-bot[bot] merged 3 commits intorapidsai:branch-24.12from
hcho3:optional_cuvs

Conversation

@hcho3
Copy link
Copy Markdown
Contributor

@hcho3 hcho3 commented Oct 28, 2024

cuVS should not be required when building cuML with CUML_ALGORITHMS=fil.

@hcho3 hcho3 requested a review from a team as a code owner October 28, 2024 21:06
@hcho3 hcho3 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 31, 2024
@hcho3
Copy link
Copy Markdown
Contributor Author

hcho3 commented Nov 1, 2024

/merge

@hcho3
Copy link
Copy Markdown
Contributor Author

hcho3 commented Nov 1, 2024

@dantegd Can you approve it? It's ready to be merged

@rapids-bot rapids-bot Bot merged commit a0186c7 into rapidsai:branch-24.12 Nov 4, 2024
@dantegd
Copy link
Copy Markdown
Member

dantegd commented Nov 4, 2024

/merge

@hcho3 hcho3 deleted the optional_cuvs branch November 4, 2024 23:07
wphicks pushed a commit to wphicks/cuml that referenced this pull request Nov 25, 2024
cuVS should not be required when building cuML with `CUML_ALGORITHMS=fil`.

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - William Hicks (https://github.com/wphicks)
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#6125
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake CUDA/C++ 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.

3 participants