Skip to content

DOC Ensures that laplacian_kernel passes numpydoc validation#23005

Merged
thomasjpfan merged 5 commits intoscikit-learn:mainfrom
gustavo-ren:issue-21350-laplacian-kernel
Apr 4, 2022
Merged

DOC Ensures that laplacian_kernel passes numpydoc validation#23005
thomasjpfan merged 5 commits intoscikit-learn:mainfrom
gustavo-ren:issue-21350-laplacian-kernel

Conversation

@gustavo-ren
Copy link
Copy Markdown
Contributor

Reference Issues/PRs
Addresses #21350

@jeremiedbb
Copy link
Copy Markdown
Member

Thanks for the PR @gustavo-ren. I made some suggestions.

@jeremiedbb jeremiedbb added No Changelog Needed Quick Review For PRs that are quick to review labels Mar 31, 2022
gustavo-ren and others added 2 commits April 1, 2022 08:42
Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@gustavo-ren gustavo-ren left a comment

Choose a reason for hiding this comment

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

Great suggestions, thank you!

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
Copy link
Copy Markdown
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @gustavo-ren

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.

LGTM

@thomasjpfan thomasjpfan merged commit 231dd7b into scikit-learn:main Apr 4, 2022
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Apr 6, 2022
…learn#23005)

* laplacian_kernel docstring correction on issue scikit-learn#21350

* Update sklearn/metrics/pairwise.py

Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

* Update sklearn/metrics/pairwise.py

Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

* Update sklearn/metrics/pairwise.py

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
jjerphan pushed a commit to jjerphan/scikit-learn that referenced this pull request Apr 29, 2022
…learn#23005)

* laplacian_kernel docstring correction on issue scikit-learn#21350

* Update sklearn/metrics/pairwise.py

Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

* Update sklearn/metrics/pairwise.py

Adding sugestiong by jeremiedbb

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

* Update sklearn/metrics/pairwise.py

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.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