Skip to content

[WIP] Fix batched silhouette score reduction aliasing - #2422

Draft
csadorf wants to merge 4 commits into
NVIDIA:mainfrom
csadorf:fix-batched-silhouette-reduction-aliasing
Draft

[WIP] Fix batched silhouette score reduction aliasing#2422
csadorf wants to merge 4 commits into
NVIDIA:mainfrom
csadorf:fix-batched-silhouette-reduction-aliasing

Conversation

@csadorf

@csadorf csadorf commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Stores the row-wise minimum for batched silhouette scores in a separate device vector, preventing the reduction output from overwriting matrix elements that may still be read.

@csadorf csadorf added bug Something isn't working non-breaking Introduces a non-breaking change labels Aug 6, 2026
@copy-pr-bot

copy-pr-bot Bot commented Aug 6, 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.

@copy-pr-bot

copy-pr-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@csadorf

csadorf commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test f8c9939

@csadorf
csadorf force-pushed the fix-batched-silhouette-reduction-aliasing branch from f8c9939 to 6ec7aeb Compare August 6, 2026 18:08
@csadorf

csadorf commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 6ec7aeb

@csadorf csadorf changed the title Fix batched silhouette score reduction aliasing [WIP] Fix batched silhouette score reduction aliasing Aug 6, 2026
@csadorf

csadorf commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 6ec7aeb

@csadorf
csadorf force-pushed the fix-batched-silhouette-reduction-aliasing branch from 6ec7aeb to d24aff2 Compare August 7, 2026 14:57
@csadorf

csadorf commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test d24aff2

@csadorf
csadorf force-pushed the fix-batched-silhouette-reduction-aliasing branch from d24aff2 to 77aaa23 Compare August 7, 2026 15:48
@csadorf

csadorf commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 77aaa23

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.

1 participant