Skip to content

DOC Ensures that load_linnerud passes numpydoc validation#22484

Merged
thomasjpfan merged 4 commits intoscikit-learn:mainfrom
DennisOsei:numpydoc-validation
Feb 27, 2022
Merged

DOC Ensures that load_linnerud passes numpydoc validation#22484
thomasjpfan merged 4 commits intoscikit-learn:mainfrom
DennisOsei:numpydoc-validation

Conversation

@DennisOsei
Copy link
Copy Markdown
Contributor

Reference Issues/PRs

Addresses #21350

What does this implement/fix? Explain your changes.

  1. Removed sklearn.datasets._base.load_linnerud from function_docstring_ignore_list.
  2. Added description to (data, target).

Any other comments?

Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
@DennisOsei DennisOsei changed the title DOC Ensures that sklearn.datasets._base.load_linnerud passes numpydoc validation DOC Ensures that linnerud passes numpydoc validation Feb 19, 2022
@DennisOsei
Copy link
Copy Markdown
Contributor Author

@glemaitre this PR is still open. Hopefully you haven't forgotten about it 😄

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.

Thanks for the PR @DennisOsei !

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
@DennisOsei
Copy link
Copy Markdown
Contributor Author

Thanks @thomasjpfan, commited the suggested changes.

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.

Minor comment otherwise LGTM

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
@DennisOsei
Copy link
Copy Markdown
Contributor Author

@thomasjpfan updated.

@thomasjpfan thomasjpfan merged commit bdd5a9e into scikit-learn:main Feb 27, 2022
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Mar 1, 2022
)

Co-authored-by: Guillaume Lemaitre <g.lemaitre58@gmail.com>
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
@thomasjpfan thomasjpfan changed the title DOC Ensures that linnerud passes numpydoc validation DOC Ensures that load_linnerud passes numpydoc validation Mar 12, 2022
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