Skip to content

implemented hierarchal clustering#312

Closed
DanielLacina wants to merge 3 commits intosmartcorelib:developmentfrom
DanielLacina:hierarchal
Closed

implemented hierarchal clustering#312
DanielLacina wants to merge 3 commits intosmartcorelib:developmentfrom
DanielLacina:hierarchal

Conversation

@DanielLacina
Copy link
Copy Markdown
Contributor

No description provided.

@DanielLacina DanielLacina requested a review from Mec-iS as a code owner June 25, 2025 01:00
@DanielLacina
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

My Python implementation works perfectly.

@Mec-iS
Copy link
Copy Markdown
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

Comment thread src/cluster/mod.rs
//! 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

thanks

@DanielLacina
Copy link
Copy Markdown
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