mpadge / spatialcluster

spatially-constrained clustering in R
https://mpadge.github.io/spatialcluster/
30 stars 6 forks source link

Fix use of distances in all algorithms #27

Closed mpadge closed 2 years ago

mpadge commented 2 years ago

Current code mixes both spatial and data-based distance metrics. The algorithms actually require all initial processing steps to be conducted with the spatial distances only, and then the tree partitioning to use the data-based distances. That's the reason for current messes like https://github.com/mpadge/spatialcluster/issues/26#issuecomment-1307523058

mpadge commented 2 years ago

@geniusadventurer fixing this issue should also yield sensible results with your data. I'll post an updated plot output of your results here.

mpadge commented 2 years ago

This looks much better:

geniusadventurer commented 2 years ago

Yes. It looks very good.