Skip to content

DOC Ensures that assert_all_finite passes numpydoc validation#22470

Merged
glemaitre merged 3 commits intoscikit-learn:mainfrom
UgoM:UM-docstring-assert_all_finite
Feb 14, 2022
Merged

DOC Ensures that assert_all_finite passes numpydoc validation#22470
glemaitre merged 3 commits intoscikit-learn:mainfrom
UgoM:UM-docstring-assert_all_finite

Conversation

@UgoM
Copy link
Copy Markdown
Contributor

@UgoM UgoM commented Feb 13, 2022

Addresses #21350

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 @UgoM !

I left a minor comment, otherwise LGTM

Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
@glemaitre
Copy link
Copy Markdown
Member

I accepted the small suggestion of @thomasjpfan since this is tiny and the PR is good to be merged. Thanks @UgoM

@glemaitre glemaitre merged commit 9712e80 into scikit-learn:main Feb 14, 2022
@UgoM UgoM deleted the UM-docstring-assert_all_finite branch February 14, 2022 11:37
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request Mar 1, 2022
…-learn#22470)

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

3 participants