Skip to content

Correcting description of test_size parameter in model_selection.GroupShuffleSplit#13414

Merged
jnothman merged 1 commit intoscikit-learn:masterfrom
OFlanagan:CorrectingGroupShuffleSplitDocumentation
Mar 12, 2019
Merged

Correcting description of test_size parameter in model_selection.GroupShuffleSplit#13414
jnothman merged 1 commit intoscikit-learn:masterfrom
OFlanagan:CorrectingGroupShuffleSplitDocumentation

Conversation

@OFlanagan
Copy link
Copy Markdown
Contributor

@OFlanagan OFlanagan commented Mar 8, 2019

Reference Issues/PRs

Fixes issue (1) in #13369

What does this implement/fix? Explain your changes.

Corrected docstring to reflect that if the test_size parameter is passed an int, it represents the absolute number of test groups. This is in line with the rest of the documentation.

Any other comments?

…assed an int, it represents the absolute number of test groups. This is in line with the rest of the documentation.
@OFlanagan OFlanagan changed the title Correcting description of test_sting parameter in model_selection.GroupShuffleSplit Correcting description of test_size parameter in model_selection.GroupShuffleSplit Mar 8, 2019
@jnothman jnothman merged commit f4e5224 into scikit-learn:master Mar 12, 2019
@jnothman
Copy link
Copy Markdown
Member

Thanks @OFlanagan!

@OFlanagan OFlanagan deleted the CorrectingGroupShuffleSplitDocumentation branch March 13, 2019 07:16
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
koenvandevelde pushed a commit to koenvandevelde/scikit-learn that referenced this pull request Jul 12, 2019
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.

2 participants