Skip to content

Conversation

rmcgibbo
Copy link
Contributor

@rmcgibbo rmcgibbo commented May 2, 2014

The order of the return values currently listed in the cross_validation._fit_and_score docstring does not match the actual order of the return values when the kwarg return_train_score=True. In the code, this option causes the train_score to be the first return value, but in the docstring it's implied that it's the second.

This PR changes the docstring to be in sync with the code.

…docstring to reflect what the code actually does
@rmcgibbo
Copy link
Contributor Author

rmcgibbo commented May 2, 2014

This was mentioned before, right after #2736 was merged https://github.com/scikit-learn/scikit-learn/pull/2736/files#r8923738

@arjoly
Copy link
Member

arjoly commented May 2, 2014

Thanks merging.

arjoly added a commit that referenced this pull request May 2, 2014
[DOC] Make _fit_and_score return value docstring consistent with code.
@arjoly arjoly merged commit 29e9ed1 into scikit-learn:master May 2, 2014
@rmcgibbo rmcgibbo deleted the _fit_and_score-docstring branch May 2, 2014 17:57
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