Skip to content

Instantiate only specific RAFT linewise kernels#1018

Merged
rapids-bot[bot] merged 18 commits intorapidsai:branch-25.08from
aamijar:raft-compile-time-invocation
Jul 2, 2025
Merged

Instantiate only specific RAFT linewise kernels#1018
rapids-bot[bot] merged 18 commits intorapidsai:branch-25.08from
aamijar:raft-compile-time-invocation

Conversation

@aamijar
Copy link
Copy Markdown
Member

@aamijar aamijar commented Jun 12, 2025

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Jun 12, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the cpp label Jun 12, 2025
@aamijar aamijar self-assigned this Jun 12, 2025
@aamijar aamijar added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jun 12, 2025
@github-actions github-actions Bot added the CMake label Jun 12, 2025
@aamijar aamijar marked this pull request as ready for review June 12, 2025 17:16
@aamijar aamijar requested review from a team as code owners June 12, 2025 17:16
Comment thread cpp/cmake/thirdparty/get_raft.cmake Outdated
Comment thread cpp/src/cluster/detail/kmeans.cuh Outdated
Copy link
Copy Markdown
Contributor

@tarang-jain tarang-jain left a comment

Choose a reason for hiding this comment

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

LGTM overall. CI is also passing so that is great. However, can you add the DO NOT MERGE tag to this PR until the raft PR #2701 is merged? (And also, a reminder to revert the changes to get_raft.cmake once the raft PR is merged)
Furthermore, can you report the libcuvs.so binary size reduction due to this change once the raft PR is merged?

@aamijar
Copy link
Copy Markdown
Member Author

aamijar commented Jul 1, 2025

Yes, updated the PR description and added the tag

@github-actions github-actions Bot removed the CMake label Jul 2, 2025
@divyegala
Copy link
Copy Markdown
Member

/merge

@rapids-bot rapids-bot Bot merged commit 3fcaf8d into rapidsai:branch-25.08 Jul 2, 2025
53 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Unstructured Data Processing Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cpp improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove generic kernel invocations of MatrixLinewiseOp

3 participants