Skip to content

Conversation

SamAdamDay
Copy link
Contributor

Reference Issues/PRs

Addresses #21350

What does this implement/fix? Explain your changes.

Updated sklearn.utils.get_chunk_n_rows docstring.

  • Short summary now starts with infinitive verb.
  • Variable names are back-tick quoted.
  • The return value has a description.

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! LGTM

@thomasjpfan thomasjpfan changed the title DOC Ensures that sklearn.utils.get_chunk_n_rows passes numpydoc validation DOC Ensures that get_chunk_n_rows passes numpydoc validation Feb 18, 2022
@thomasjpfan thomasjpfan merged commit 26d218c into scikit-learn:main Feb 18, 2022
thomasjpfan pushed a commit to thomasjpfan/scikit-learn that referenced this pull request Mar 1, 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.

2 participants