The Wayback Machine - https://web.archive.org/web/20220218233852/https://github.com/scikit-learn/scikit-learn/pull/21081
Skip to content
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 Build proper sentence for Davies-Bouldin Index advantage #21081

Merged
merged 1 commit into from Sep 24, 2021

Conversation

@jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Sep 18, 2021

Build proper sentence for Davies-Bouldin Index advantage in User Guide.

Reference Issues/PRs

None.

What does this implement/fix? Explain your changes.

Build a complete, grammatically correct sentence when describing the advantages of the Davies-Boulding Index in the User Guide.

Any other comments?

None.

Build proper sentence for Davies-Bouldin Index advantage in User Guide.
Copy link
Contributor

@glemaitre glemaitre left a comment

LGTM

@ogrisel ogrisel merged commit 9b90126 into scikit-learn:main Sep 24, 2021
33 checks passed
@ogrisel
Copy link
Member

@ogrisel ogrisel commented Sep 24, 2021

Thanks for the fix!

@jhlegarreta jhlegarreta deleted the FixDBIndexDocUserGuide branch Sep 24, 2021
@glemaitre glemaitre mentioned this pull request Oct 23, 2021
10 tasks
glemaitre added a commit to glemaitre/scikit-learn that referenced this issue Oct 23, 2021
…learn#21081)

Build proper sentence for Davies-Bouldin Index advantage in User Guide.
glemaitre added a commit that referenced this issue Oct 25, 2021
Build proper sentence for Davies-Bouldin Index advantage in User Guide.
samronsin added a commit to samronsin/scikit-learn that referenced this issue Nov 30, 2021
…learn#21081)

Build proper sentence for Davies-Bouldin Index advantage in User Guide.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants