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

Alter MutualInfoCalculatorMultiVariate significance calculations to specify which variable should be reordered #4

Open
GoogleCodeExporter opened this issue Jul 23, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

The ChannelCalculators talk about reordering the source variable to evaluate 
the statistical significance, but the 
MutualInfoCalculatorMultiVariate.computeAverageLocalOfObservations(int[] 
newOrdering) method reorders the second variable (which is now the 
destination). This is because it used to have source variable as the second 
argument.

Should reorganise this code so that it follows a consistent interpretation 
(reordering the source variable - the first variable). This will necessitate 
shuffling the arguments around in the Kraskov calculators. They are currently 
consistent (MI gives right result, TE calculator uses the variables in the 
correct order) but I want this finalised.



Original issue reported on code.google.com by joseph.lizier on 6 Aug 2012 at 6:38

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

2 participants