declare cuda-python dependency for wheels, other small packaging changes #6217
+38
−27
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow-up to #6190.
Proposes some miscellaneous packaging cleanup:
cuml-cu{11,12}
wheels' runtime dependency oncuda-python
cuml/python/cuml/cuml/svm/linear.pyx
Lines 40 to 43 in bfd2e22
raft_log.txt
to.gitignore
CUML_USE_RAFT_STATIC
cuml/cpp/CMakeLists.txt
Line 600 in bfd2e22
BUILD_CAGRA_HNSWLIB OFF
inget_cuvs.cmake
cuml/cpp/cmake/thirdparty/get_raft.cmake
Line 58 in bfd2e22
ON
by default in cuVS, so this change saves a bit of build time and size: https://github.com/rapidsai/cuvs/blob/1e548f8c3a773452ce69556f4db72fc712efae02/cpp/CMakeLists.txt#L58rapids-download-wheels-from-s3
in CI scriptsNotes for Reviewers
These changes are useful independently, but will also make the
libcuml
wheels PR (#6199) a bit smaller and easier to review.