8000 [WIP] added in gaussian plot by vincentpham1991 · Pull Request #8300 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

[WIP] added in gaussian plot #8300

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

Closed
wants to merge 5 commits into from

Conversation

vincentpham1991
Copy link
Contributor

Reference Issue

Addresses #2890

What does this implement/fix? Explain your changes.

Add gaussian

Any other comments?

Add in plot example of a more scattered plot (using gaussian distribution) to model a more real world example.

@vincentpham1991 vincentpham1991 changed the title added in gaussian plot [MRG] added in gaussian plot Feb 6, 2017
@vincentpham1991 vincentpham1991 changed the title [MRG] added in gaussian plot [WIP] added in gaussian plot Feb 6, 2017
Pham, Vincent (CONT) added 2 commits February 6, 2017 00:58
@@ -40,8 +40,13 @@
noise=.05)
noisy_moons = datasets.make_moons(n_samples=n_samples, noise=.05)
blobs = datasets.make_blobs(n_samples=n_samples, random_state=8)
gaussians = np.concatenate([np.multiply(np.random.randn(n_samples / 3, 2), 10),
Copy link
Member

Choose a reason for hiding this comment

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

I think writing this as

np.vstack([np.random.randn(*shape) * 10,
           np.random.randn(*shape) + 10,
           np.random.randn(*shape) - 5])

is clearer

@jnothman
Copy link
Member
jnothman commented Feb 6, 2017

Could you add GMMs to the plots to see if it can roughly find the true clusters?

Base automatically changed from master to main January 22, 2021 10:49
@thomasjpfan
Copy link
Member

Thank you for working on this. I am closing this PR because the original issue was addressed in #6305

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0