cosine similarity vs cosine distance

[tazitoo]

Wanted to say thanks for the repo - this is a giant leap forward in scaling k-medoids.

I noticed that one of the distance metrics available is cosine similarity - not cosine distance which is 1 - cosine similarity. My intuition tells me dist(me, me) should be zero, and not one.

If using cosine similarity is intentional, I could put in a PR for cos_dist.

[motiwari]

Thanks for the bug report. I have a release going out this week that will address this.

[motiwari]

Sorry for the delay. I've fixed this in 6f6de6b, which will go out in a release shortly.

[tazitoo]

Awesome - thanks for the quick fix. ...on the lam.

…

On Mar 21, 2022, at 19:42, Mo Tiwari ***@***.***> wrote:  Sorry for the delay. I've fixed this in 6f6de6b, which will go out in a release shortly. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.