Skip to content

Remove cutlass and cuco dependencies#2916

Merged
rapids-bot[bot] merged 4 commits intorapidsai:mainfrom
divyegala:rm-cutlass
Jan 7, 2026
Merged

Remove cutlass and cuco dependencies#2916
rapids-bot[bot] merged 4 commits intorapidsai:mainfrom
divyegala:rm-cutlass

Conversation

@divyegala
Copy link
Member

@divyegala divyegala commented Jan 7, 2026

As the distance APIs are now moved to cuVS, we can remove the cutlass and cuco dependencies and avoid shipping the headers.

Closes #2914

Reduces CUDA 13 libraft wheel size from 18.9M to 14.5M.

Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

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

There are remaining references to cutlass in:

  • .pre-commit-config.yaml
  • docs/source/build.md
  • cpp/include/raft/stats/detail/cov.cuh

@divyegala divyegala changed the title Remove cutlass dependency Remove cutlass and cuco dependency Jan 7, 2026
@divyegala divyegala changed the title Remove cutlass and cuco dependency Remove cutlass and cuco dependencies Jan 7, 2026
content: |
# Unspecified brings in cuco and cutlass and friends
# Unspecified brings in all components
cmake --install cpp/build --component Unspecified
Copy link
Member

@cjnolet cjnolet Jan 7, 2026

Choose a reason for hiding this comment

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

Is the user supposed to type out "Unspecified" or is this implicit? Or does cmake somehow know how to handle that?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is implicit, it just ensures the conda package has all the libraries that we ship as raft's dependencies.

rapids-bot bot pushed a commit to rapidsai/cuml that referenced this pull request Jan 7, 2026
cuML was getting its cutlass dependency from RAFT, which is now being removed in rapidsai/raft#2916. Given that cuML only references cutlass in an otherwise unused header file, we can just safely delete the file.

Authors:
  - Divye Gala (https://github.com/divyegala)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: #7655
@divyegala
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 7dbd36e into rapidsai:main Jan 7, 2026
101 checks passed
rapids-bot bot pushed a commit to rapidsai/cuvs that referenced this pull request Jan 10, 2026
#1686)

## use CUDA 13.1 devcontainers

Follow-up to #1677

There, I forgot to switch devcontainer testing here back to CUDA 13.1 (I'd temporarily kept it at 13.0 because there weren't yet NCCL packages with 13.1 support). This fixes that.

## react to cutlass removals in RAFT

rapidsai/raft#2916 removed headers used by cuVS and stopped exporting cutlass from RAFT.

This brings those headers and some related patches over here to cuVS.

Related: rapidsai/cuml#7658

Authors:
  - James Lamb (https://github.com/jameslamb)
  - Divye Gala (https://github.com/divyegala)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Robert Maynard (https://github.com/robertmaynard)
  - Dante Gama Dessavre (https://github.com/dantegd)

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

Labels

breaking Breaking change improvement Improvement / enhancement to an existing function

Development

Successfully merging this pull request may close these issues.

Remove cutlass dependency

4 participants