-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
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
base: main
Are you sure you want to change the base?
Conversation
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. |
There was a problem hiding this 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.
doc/whats_new/v1.4.rst
Outdated
- |Feature| :func:`metrics.cluster.dbcv_score` | ||
:pr:`28036` by :user:`Nils Cercariolo <Nylio-prog>`. |
There was a problem hiding this comment.
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.
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. |
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.