Skip to content

DOC Build proper sentence for Davies-Bouldin Index advantage #21081

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 1 commit into from
Sep 24, 2021

Conversation

jhlegarreta
Copy link
Contributor

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
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.

LGTM

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

ogrisel commented Sep 24, 2021

Thanks for the fix!

@jhlegarreta jhlegarreta deleted the FixDBIndexDocUserGuide branch September 24, 2021 12:32
@glemaitre glemaitre mentioned this pull request Oct 23, 2021
10 tasks
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Oct 23, 2021
…learn#21081)

Build proper sentence for Davies-Bouldin Index advantage in User Guide.
glemaitre pushed a commit that referenced this pull request Oct 25, 2021
Build proper sentence for Davies-Bouldin Index advantage in User Guide.
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request 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
Development

Successfully merging this pull request may close these issues.

3 participants