8000 [MRG+1] Fixes #8198 - error in datasets.make_moons by levy5674 · Pull Request #8199 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

[MRG+1] Fixes #8198 - error in datasets.make_moons #8199

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Jan 17, 2017
Merged

[MRG+1] Fixes #8198 - error in datasets.make_moons #8199

merged 8 commits into from
Jan 17, 2017

Conversation

levy5674
Copy link
Contributor
@levy5674 levy5674 commented Jan 14, 2017 •
edited
Loading

Reference Issue

Fixes #8198

What does this implement/fix? Explain your changes.

Inside make_moons, n_samples_out elements go into outer_circle_x and outer_circle_y but n_samples_in elements of y get set to 1

Any other comments?

%pylab inline
from sklearn.datasets import make_moons

XL, YL = make_moons(5)
XU, YU = make_moons(495)
scatter(XL[:,0], XL[:,1], c=YL, s=200)
scatter(XU[:,0], XU[:,1], c=YU)

Before:
sklearn_8198_broken

After
sklearn_8198_fixed

@GaelVaroquaux
Copy link
Member
GaelVaroquaux commented Jan 14, 2017 via email

@jnothman
Copy link
Member

LGTM. Please add a bug fix entry to whats_new.rst

@jnothman jnothman changed the title [MRG] Fixes #8198 - error in datasets.make_moons [MRG+1] Fixes #8198 - error in datasets.make_moons Jan 15, 2017
@levy5674
Copy link
Contributor Author

I believe the circleci failures are related to #8209 and will go away when #7986 is merged.