Skip to content

MNT Revert the deprecation of min_samples_leaf and min_weight_fraction_leaf#11998

Merged
rth merged 2 commits intoscikit-learn:masterfrom
jnothman:rev-dep-min_samples
Sep 8, 2018
Merged

MNT Revert the deprecation of min_samples_leaf and min_weight_fraction_leaf#11998
rth merged 2 commits intoscikit-learn:masterfrom
jnothman:rev-dep-min_samples

Conversation

@jnothman
Copy link
Copy Markdown
Member

@jnothman jnothman commented Sep 4, 2018

And improve the documentation of min_samples_leaf.

Fixes #11976

What do others think?

Copy link
Copy Markdown
Member

@amueller amueller left a comment

Choose a reason for hiding this comment

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

I agree. Sorry for pushing for this, clearly not thinking about all cases.

@amueller
Copy link
Copy Markdown
Member

amueller commented Sep 7, 2018

can we tag blockers for the final release?

@jnothman jnothman added this to the 0.20 milestone Sep 8, 2018
Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

Thanks!

@rth rth merged commit 79f5d14 into scikit-learn:master Sep 8, 2018
jnothman added a commit to jnothman/scikit-learn that referenced this pull request Sep 9, 2018
jnothman added a commit to jnothman/scikit-learn that referenced this pull request Sep 17, 2018
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.

3 participants