Skip to content

Apply ABI Breaking Changes to KMeans Params (release branch)#2329

Open
tarang-jain wants to merge 9 commits into
NVIDIA:release/26.08from
tarang-jain:breaking-26.08
Open

Apply ABI Breaking Changes to KMeans Params (release branch)#2329
tarang-jain wants to merge 9 commits into
NVIDIA:release/26.08from
tarang-jain:breaking-26.08

Conversation

@tarang-jain

Copy link
Copy Markdown
Contributor

This PR is an exact copy of #2147 but targeted to the release branch

@copy-pr-bot

copy-pr-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

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.

@tarang-jain tarang-jain self-assigned this Jul 20, 2026
@tarang-jain tarang-jain added breaking Introduces a breaking change improvement Improves an existing functionality labels Jul 20, 2026
@tarang-jain
tarang-jain marked this pull request as ready for review July 20, 2026 22:04
@tarang-jain
tarang-jain requested review from a team as code owners July 20, 2026 22:04
@tarang-jain
tarang-jain requested a review from a team as a code owner July 20, 2026 22:20
@tarang-jain
tarang-jain requested a review from jameslamb July 20, 2026 22:20
rapids-bot Bot pushed a commit that referenced this pull request Jul 23, 2026
This PR renames the `streaming_batch_size` param for kmeans to:
1. Better convey the meaning of the param -- link to discussion: #2323 (comment)
2. Avoid confusion with the already existing `batch_samples` and `batch_centroids` params

The idea is that the `streaming` word can be misleading. We are "buffering up" a batch of samples onto the device.

Merge after #2329

Authors:
  - Tarang Jain (https://github.com/tarang-jain)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: #2328
@tarang-jain

Copy link
Copy Markdown
Contributor Author

/ok to test ddc7040

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

Labels

breaking Introduces a breaking change improvement Improves an existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants