search

BianKov / iterEmb

Iterative Embedding and ReWeighting (IERW)
1 stars 0 forks source link

Why elkan? #11

Closed skojaku closed 1 year ago

skojaku commented 1 year ago

https://github.com/BianKov/iterEmb/blob/cbcb87e7ec6ae92c01f91748ced4cf365ee7abbe/iteremb/iteremb/communityDetection.py#L186

Why not the default lloyd algorithm?

BianKov commented 1 year ago

I think simply because on most of my computers I have an older version of scikit-learn and the lloyd became the default only in version 1.1, so I fixed the algorithm parameter to the default of most of my computers. Maybe it doesn't make any sense. I don't remember any other reason why I did this. You can change to lloyd if you want. Basically I just wanted to clearly specify the algorithm parameter because its default changes between the different versions of scikit-learn. But it can be set to lloyd if its better.

skojaku commented 1 year ago

Thanks! That clarified.