Skip to content

implemented hierarchal clustering #312

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 3 commits into
base: development
Choose a base branch
from

Conversation

DanielLacina
Copy link
Contributor

No description provided.

@DanielLacina DanielLacina requested a review from Mec-iS as a code owner June 25, 2025 01:00
@DanielLacina
Copy link
Contributor Author

Whenever I create a certain n amount of clusters, the labels returned by this implementation and sklearn start to deviate. Is it because of the difference in numeric precision with the Python ml libraries?

@DanielLacina
Copy link
Contributor Author

My Python implementation works perfectly.

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jun 25, 2025

I will take a look. For now you can compare with this implementation https://github.com/rom1mouret/catclustering

@@ -4,5 +4,6 @@
//! are more similar to other data points in the same group than those in other groups. In simple words, the aim is to segregate groups with similar traits and assign them into clusters.

pub mod dbscan;
pub mod hierarchal;
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo: hierarchical

also better to use agglomerative

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jun 25, 2025

@DanielLacina as a pairing algorithm please use fastpair as it is implemented in the pairing module

@DanielLacina DanielLacina reopened this Jun 25, 2025
@DanielLacina
Copy link
Contributor Author

thanks

@DanielLacina
Copy link
Contributor Author

Do you think we should add a linkage attribute to the FastPair struct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants