8000 DOC - Ensures that GridSearchCV passes numpydoc validation by EricEllwanger · Pull Request #21003 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

DOC - Ensures that GridSearchCV passes numpydoc validation #21003

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Sep 13, 2021

Conversation

EricEllwanger
Copy link
Contributor

Reference Issues/PRs

Addresses #20308

What does this implement/fix? Explain your changes.

Ensure GridSearchCV passes numpydoc validation by making changes to sklearn/model_selection/search.py:
GridSearchCV
GridSearchCV.classes

GridSearchCV.decision_function
GridSearchCV.fit
GridSearchCV.inverse_transform
GridSearchCV.n_features_in_
GridSearchCV.predict
GridSearchCV.predict_log_proba
GridSearchCV.predict_proba
GridSearchCV.score
GridSearchCV.score_samples
GridSearchCV.transform

Any other comments?

…ion - Corrected sections and estimator parameter
…c validation - added Returns section description
@glemaitre glemaitre self-requested a review September 13, 2021 08:00
Copy link
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.

I made a push to correct the indentation since this is not really easy to propose the change in GitHub. I will merge when the CIs are green.

Thanks @EricEllwanger

@glemaitre glemaitre merged commit 28ef997 into scikit-learn:main Sep 13, 2021
@EricEllwanger EricEllwanger deleted the doc-fixes branch September 13, 2021 12:21
adrinjalali pushed a commit to adrinjalali/scikit-learn that referenced this pull request Sep 20, 2021
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 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
0