Skip to content

DOC Ensures that function passes numpydoc validation: has_fit_parameter#21590

Merged
jjerphan merged 3 commits intoscikit-learn:mainfrom
reshamas:np_validation
Nov 15, 2021
Merged

DOC Ensures that function passes numpydoc validation: has_fit_parameter#21590
jjerphan merged 3 commits intoscikit-learn:mainfrom
reshamas:np_validation

Conversation

@reshamas
Copy link
Copy Markdown
Member

@reshamas reshamas commented Nov 7, 2021

Reference Issues/PRs

References #21350.
Closes #21471.

What does this implement/fix? Explain your changes.

Updates docstring in function to fix numpydoc errors.

Any other comments?

#DataUmbrella sprint

Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

LGTM otherwise.

"sklearn.utils.validation.check_random_state",
"sklearn.utils.validation.column_or_1d",
"sklearn.utils.validation.has_fit_parameter",
# "sklearn.utils.validation.has_fit_parameter",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You can remove this line then.

Copy link
Copy Markdown
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you, @reshamas.

@jjerphan jjerphan merged commit d40943a into scikit-learn:main Nov 15, 2021
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Nov 22, 2021
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Nov 29, 2021
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Dec 24, 2021
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