Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Ensures that sklearn.metrics._classification.log_loss passes numpydoc validation #23657

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

Merged
merged 7 commits into from
Jun 22, 2022

Conversation

paulo-smcs
Copy link
Contributor

Reference Issues/PRs

Adress #21350

What does this implement/fix? Explain your changes.

Make sure that the docstring passes the numpydoc test.

Any other comments?

To pass the test I had to add a description of the returned value of the log_loss function. I am not sure if my description, was the most appropriate one. If would by nice to check it. Thanks!

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

@paulo-smcs
Copy link
Contributor Author

Reversed the changes keeping the commit. Don't know if this is a good practise.

@jeremiedbb
Copy link
Member

Reversed the changes keeping the commit. Don't know if this is a good practise.

Not an issue in scikit-learn since we squash the commits before merging anyway

Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

Thanks @paulo-smcs

@jeremiedbb jeremiedbb merged commit cf7f681 into scikit-learn:main Jun 22, 2022
ogrisel pushed a commit to ogrisel/scikit-learn that referenced this pull request Jul 11, 2022
…ydoc validation (scikit-learn#23657)

Co-authored-by: Thomas J. Fan <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Aug 4, 2022
…ydoc validation (scikit-learn#23657)

Co-authored-by: Thomas J. Fan <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
glemaitre pushed a commit that referenced this pull request Aug 5, 2022
…ydoc validation (#23657)

Co-authored-by: Thomas J. Fan <[email protected]>
Co-authored-by: Jérémie du Boisberranger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants