Skip to content

add dbcv score (unsupervised clustering score) and one test comparing kmeans and dbscan dbcv score #28036

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Nylio-prog
Copy link

@Nylio-prog Nylio-prog commented Dec 30, 2023

Reference Issues/PRs

Fixes #27259

What does this implement/fix? Explain your changes.

This adds the dbcv score which is a density based score for unsupervised clustering. I've also included one test that compares kmeans and dbscan's dbcv scores with the moon dataset. This follows this paper.

Any other comments?

I wasn't sure where to add all the helper functions for dbcv score as it is a bit more complex than other metrics so I added another file but feel free to propose other options as it is my first PR. The implementation follows closely this repo.

Copy link

github-actions bot commented Dec 31, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 99b3e25. Link to the linter CI: here

@Nylio-prog
Copy link
Author

Nylio-prog commented Dec 31, 2023

I only added the new dbcv_score into the changelog v1.4 as a feature, tell me if I should also add the helper functions.

Copy link
Member

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

I don't see any decisions made on the original issue regarding what we want to add, and whether we want to add this. So this needs to wait till we make a decision there.

Comment on lines 610 to 611
- |Feature| :func:`metrics.cluster.dbcv_score`
:pr:`28036` by :user:`Nils Cercariolo <Nylio-prog>`.
Copy link
Member

Choose a reason for hiding this comment

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

needs to be in the 1.5 changelog.

@adrinjalali adrinjalali added Needs Decision Requires decision Needs Decision - Include Feature Requires decision regarding including feature labels Jan 2, 2024
@adrinjalali
Copy link
Member

This is not really a good first issue to try really. Please try one of the issues labeled as good first issue to get used to the workflow here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module:metrics Needs Decision - Include Feature Requires decision regarding including feature Needs Decision Requires decision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New clustering metrics
2 participants