Skip to content

DOC Add link and fix typo in nearest neighbours estimators#17017

Merged
thomasjpfan merged 1 commit intoscikit-learn:masterfrom
lucyleeow:DOC_neigh_typo
Apr 23, 2020
Merged

DOC Add link and fix typo in nearest neighbours estimators#17017
thomasjpfan merged 1 commit intoscikit-learn:masterfrom
lucyleeow:DOC_neigh_typo

Conversation

@lucyleeow
Copy link
Copy Markdown
Member

@lucyleeow lucyleeow commented Apr 23, 2020

Reference Issues/PRs

None

What does this implement/fix? Explain your changes.

Add same link and fix same typo in:
NearestNeighbors
RadiusNeighborsRegressor
KNeighborsClassifier
RadiusNeighborsClassifier

Any other comments?

Happy to split into separate PRs if required.

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

@thomasjpfan thomasjpfan merged commit 3693492 into scikit-learn:master Apr 23, 2020
@lucyleeow lucyleeow deleted the DOC_neigh_typo branch May 12, 2020 18:36
gio8tisu pushed a commit to gio8tisu/scikit-learn that referenced this pull request May 15, 2020
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
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