Clustering.jl icon indicating copy to clipboard operation
Clustering.jl copied to clipboard

varinfo() clashes with InteractiveUtils.varinfo()

Open alyst opened this issue 5 years ago • 3 comments

Unless there are better ideas, I suggest to rename it into variatinfo(), because varinfo really sounds like some information about a variable. What should be the roadmap for renaming? Introduce the new name and deprecate varinfo() in the next minor release, then remove varinfo() after some period of time (6 month or so)?

cc @ararslan

alyst avatar Aug 14 '18 09:08 alyst

Hm. I'm not crazy about variatinfo. Perhaps something like infovariation or even sharedinfodist?

ararslan avatar Aug 14 '18 15:08 ararslan

infovariation 👍

wildart avatar Aug 14 '18 21:08 wildart

sharedinfodist :+1: I'd rather avoid abbreviation, so what about sharedinfodistance or sharedinfometric?

I was also thinking about infovariation, but IMHO "shared info metric/distance" describes this function much better than "variation of information" (I find this choice of the term rather arbitrary), and I'm not sure we want to promote yet another vague generic name ("information variation") for this function.

alyst avatar Aug 15 '18 20:08 alyst