-
Notifications
You must be signed in to change notification settings - Fork 43
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
Warning in kmedoids.fit when using seuclidian metric #113
Comments
This is about scikit-learn sklearn.metrics.pairwise and not about kmedoids. |
Thanks for your fast reply. Yes, strictly speaking it is about sklearn.metrics.pairwise and not kmedoids but how will I be able to pass V as an argument or can V be computed internally by Kmedoids? |
Ah, you are right.
|
I think we should handle this in scikit-learn-extra. So that passing |
I would be very happy if scikit-learn-extra could take care of that instead of having to do some gimmicks myself. |
Thank you so much! |
When I use kmedoids.fit() with the 'seuclidian' distance (computed internally by sklearn.metrics.pairwise using scipy.spatial.distance), I get the following warning:
from version 1.0 (renaming of 0.25), pairwise_distances for metric='seuclidean' will require V to be specified if Y is passed.
Should I be worried? Is the metric computed correctly?
Thanks in advance for your time!
The text was updated successfully, but these errors were encountered: