-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
DOC Fixes numpydoc validation errors in KNeighborsClassifier #20373
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
DOC Fixes numpydoc validation errors in KNeighborsClassifier #20373
Conversation
I think there's a linting issue with a line being too long, I think it might be whitespace? |
…ressor description on the See Also section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Let see if the CIs agree ;)
There was a problem hiding this 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 @pibieta !
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
It seems I need an approval for github to run the test on 2 worlkflows. Can you help with that @glemaitre ? |
Looks good. Merging. Thanks @pibieta |
…learn#20373) Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
Reference Issues/PRs
Addresses #20308
What does this implement/fix? Explain your changes.
Fixes numpydoc validation errors in KNeighborsClassifier, namely:
See Also
descriptions.Any other comments?
#DataUmbrella sprint
cc: @pibieta @g4brielvs