Skip to content

[MRG+1] DOCATHON : Provide more intuition on perplexity in the documentation,… #8551

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 8, 2017

Conversation

lmcinnes
Copy link
Contributor

@lmcinnes lmcinnes commented Mar 7, 2017

Trying to resolve #8546

Added some more discussion on the intuition behind perplexity, and when and why you may wish to increase it. Also linked to the "How to Use t-SNE Effectively" page.

@@ -554,6 +561,10 @@ is a tradeoff between performance and accuracy. Larger angles imply that we
can approximate larger regions by a single point,leading to better speed
but less accurate results.

`"How to Use t_SNE Effectively" <http://distill.pub/2016/misread-tsne/>`_
Copy link
Member

Choose a reason for hiding this comment

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

That blog post is really nice!

Copy link
Member

@NelleV NelleV left a comment

Choose a reason for hiding this comment

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

It looks good to me. I think this is a great improvement on what we currently have!

@NelleV NelleV changed the title DOCATHON : Provide more intuition on perplexity in the documentation,… [MRG+1] DOCATHON : Provide more intuition on perplexity in the documentation,… Mar 7, 2017
Copy link
Contributor

@rishikksh20 rishikksh20 left a comment

Choose a reason for hiding this comment

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

LGTM

@lesteve
Copy link
Member

lesteve commented Mar 8, 2017

LGTM, merging, thanks a lot!

@lesteve lesteve merged commit 425bf83 into scikit-learn:master Mar 8, 2017
@Przemo10 Przemo10 mentioned this pull request Mar 17, 2017
herilalaina pushed a commit to herilalaina/scikit-learn that referenced this pull request Mar 26, 2017
massich pushed a commit to massich/scikit-learn that referenced this pull request Apr 26, 2017
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
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.

T-SNE docs don't explain perplexity well
4 participants