-
Notifications
You must be signed in to change notification settings - Fork 26
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
KMeansClustering no longer works with custom objects. #15
Comments
With the current implementation of
It would make more sense to enforce non-tuple data elements to be a subclass of an ABC which has the above methods as abstract. Where I will follow semantic versioning, and as this would change the external API, I will postpone this for |
This should probably get dropped in favour of subclassing data-elements (see #15)
I believe HierarchicalClustering also does not work with custom objects e.g. I have following code:
where, I am getting this error trace:
|
Thanks for raising this. I have to admit that this package has fallen a bit off my radar since I moved jobs a couple of years ago. I'll try to find some time to work on this. The package certainly could do with some love again... :) |
Sure, is there a quick hacky fix for this issue? which I can do and get unblocked. |
Sorry for the late reply... I wrote the answer above right before hitting the sack. I'll see if I can find something. |
The following code gives an error:
The error:
The text was updated successfully, but these errors were encountered: