Skip to content

DOC Fix doc of default values in sklearn.neural_network._multilayer_perceptron.py#18133

Merged
thomasjpfan merged 2 commits intoscikit-learn:masterfrom
martina-zxy:doc_mlp
Aug 9, 2020
Merged

DOC Fix doc of default values in sklearn.neural_network._multilayer_perceptron.py#18133
thomasjpfan merged 2 commits intoscikit-learn:masterfrom
martina-zxy:doc_mlp

Conversation

@martina-zxy
Copy link
Copy Markdown
Contributor

Reference Issues/PRs

See #15761

What does this implement/fix? Explain your changes.

This PR update docstring in sklearn.neural_network._multilayer_perceptron.py according to the guideline.

Any other comments?

Copy link
Copy Markdown
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 @martina-zxy !

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
Copy link
Copy Markdown
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.

LGTM

@thomasjpfan thomasjpfan merged commit e400fa8 into scikit-learn:master Aug 9, 2020
jayzed82 pushed a commit to jayzed82/scikit-learn that referenced this pull request Oct 22, 2020
…erceptron.py (scikit-learn#18133)

* DOC Fix doc of default values in sklearn.neural_network._multilayer_perceptron.py

* Update sklearn/neural_network/_multilayer_perceptron.py

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
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.

2 participants