Skip to content

Fix batched silhouette score stream ordering#2316

Merged
rapids-bot[bot] merged 2 commits into
NVIDIA:mainfrom
csadorf:issue-8145-ci-test-silhouette-score-batched-fails-for-l1-metr
Jul 10, 2026
Merged

Fix batched silhouette score stream ordering#2316
rapids-bot[bot] merged 2 commits into
NVIDIA:mainfrom
csadorf:issue-8145-ci-test-silhouette-score-batched-fails-for-l1-metr

Conversation

@csadorf

@csadorf csadorf commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Runs each pairwise-distance computation and its consumer on the same CUDA stream by binding each batch chunk to a stream-specific RAFT handle. Adds regression coverage using a four-stream pool and repeated comparisons against the non-batched implementation.

Closes #2314

@csadorf csadorf requested a review from a team as a code owner July 10, 2026 16:45
@csadorf csadorf added bug Something isn't working non-breaking Introduces a non-breaking change labels Jul 10, 2026
@csadorf csadorf force-pushed the issue-8145-ci-test-silhouette-score-batched-fails-for-l1-metr branch from 2d239c0 to ffe70c9 Compare July 10, 2026 16:54
@csadorf

csadorf commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot rapids-bot Bot merged commit 772de40 into NVIDIA:main Jul 10, 2026
163 of 165 checks passed
@csadorf csadorf deleted the issue-8145-ci-test-silhouette-score-batched-fails-for-l1-metr branch July 10, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Batched silhouette score has a stream-ordering race

2 participants