Skip to content

Conversation

yuanx749
Copy link
Contributor

Reference Issues/PRs

#21598

What does this implement/fix? Explain your changes.

Reduce from 18 sec to 7 sec by halving the number of samples and reducing n_iter from default 1000 to 300 or 400.
Also set learning_rate="auto" to disable FutureWarning.

Before:
before

After:
after

Any other comments?

Copy link
Member

@TomDLT TomDLT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

@TomDLT TomDLT mentioned this pull request Nov 11, 2021
41 tasks
@adrinjalali adrinjalali merged commit f695147 into scikit-learn:main Nov 16, 2021
@yuanx749 yuanx749 deleted the plot_t_sne_perplexity branch November 16, 2021 14:37
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Nov 22, 2021
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Nov 29, 2021
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Dec 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants