Skip to content
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

varinfo() clashes with InteractiveUtils.varinfo() #123

Open
alyst opened this issue Aug 14, 2018 · 3 comments
Open

varinfo() clashes with InteractiveUtils.varinfo() #123

alyst opened this issue Aug 14, 2018 · 3 comments

Comments

@alyst
Copy link
Member

alyst commented Aug 14, 2018

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

@ararslan
Copy link
Member

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

@wildart
Copy link
Contributor

wildart commented Aug 14, 2018

infovariation 👍

@alyst
Copy link
Member Author

alyst commented Aug 15, 2018

sharedinfodist 👍 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants