Skip to content

Fix UMAP transform#6449

Merged
rapids-bot[bot] merged 6 commits intorapidsai:branch-25.04from
viclafargue:fix-umap-transform
Mar 19, 2025
Merged

Fix UMAP transform#6449
rapids-bot[bot] merged 6 commits intorapidsai:branch-25.04from
viclafargue:fix-umap-transform

Conversation

@viclafargue
Copy link
Copy Markdown
Contributor

The number of embeddings fitted during the training step (tail and tail_embeddings) was set incorrectly when processing the transformation step. This impacted negative sampling and in turn the trustworthiness score.

@viclafargue viclafargue requested a review from a team as a code owner March 17, 2025 18:36
@viclafargue viclafargue requested review from cjnolet and dantegd March 17, 2025 18:36
@viclafargue viclafargue requested a review from a team as a code owner March 18, 2025 14:59
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label Mar 18, 2025
@jcrist jcrist added bug Something isn't working non-breaking Non-breaking change labels Mar 18, 2025
@csadorf csadorf mentioned this pull request Mar 18, 2025
9 tasks
@jcrist
Copy link
Copy Markdown
Member

jcrist commented Mar 19, 2025

/merge

@rapids-bot rapids-bot Bot merged commit 0b13f70 into rapidsai:branch-25.04 Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CUDA/C++ Cython / Python Cython or Python issue non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants