JordanAsh / badge

An implementation of the BADGE batch active learning algorithm.
197 stars 32 forks source link

Repetitive indices in badge_sampling.py init_centers function #5

Open AIasd opened 3 years ago

AIasd commented 3 years ago

I notice that there is no mechanism to avoid repetition of indices in the indsAll returned in the end.

JordanAsh commented 3 years ago

Good find! Yes, I actually have that changed on my end but haven't pushed to git yet. I'll do so as soon as I get a moment. That said, barring numerical issues, k-means++ should never repeat an index.

On Tue, Feb 2, 2021 at 11:19 PM ZiyuanZhong notifications@github.com wrote:

I notice that there is no mechanism to avoid repetition of indices in the indsAll returned in the end.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/JordanAsh/badge/issues/5, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEAIDG5WAB7TCXSA33UIHHLS5DFDVANCNFSM4XAEAK4Q .