Skip to content

Map Barnes-Hut to FFT for T-SNE in cuml.accel.#6619

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-25.06from
csadorf:fix/switch-tsne-cuml.accel-default-method-to-fft
May 8, 2025
Merged

Map Barnes-Hut to FFT for T-SNE in cuml.accel.#6619
rapids-bot[bot] merged 4 commits intorapidsai:branch-25.06from
csadorf:fix/switch-tsne-cuml.accel-default-method-to-fft

Conversation

@csadorf
Copy link
Copy Markdown
Contributor

@csadorf csadorf commented May 5, 2025

Mitigation for #6617.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 5, 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 Cython / Python Cython or Python issue label May 5, 2025
@csadorf csadorf linked an issue May 5, 2025 that may be closed by this pull request
@csadorf csadorf added bug Something isn't working breaking Breaking change labels May 5, 2025
@csadorf
Copy link
Copy Markdown
Contributor Author

csadorf commented May 5, 2025

/ok to test 6d2eb6b

Comment thread python/cuml/cuml/manifold/t_sne.pyx
@jcrist jcrist marked this pull request as ready for review May 7, 2025 21:45
@jcrist jcrist requested a review from a team as a code owner May 7, 2025 21:45
@jcrist jcrist requested review from betatim and jcrist May 7, 2025 21:45
@csadorf
Copy link
Copy Markdown
Contributor Author

csadorf commented May 7, 2025

/merge

@csadorf
Copy link
Copy Markdown
Contributor Author

csadorf commented May 8, 2025

/merge

@rapids-bot rapids-bot Bot merged commit ebcdc34 into rapidsai:branch-25.06 May 8, 2025
88 checks passed
@csadorf csadorf deleted the fix/switch-tsne-cuml.accel-default-method-to-fft branch May 8, 2025 02:32
Ofek-Haim pushed a commit to Ofek-Haim/cuml that referenced this pull request May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change bug Something isn't working Cython / Python Cython or Python issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change the T-SNE default method to FFT for cuml.accel

3 participants