dwuab / information-dynamics-toolkit

Automatically exported from code.google.com/p/information-dynamics-toolkit
GNU General Public License v3.0
0 stars 0 forks source link

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

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
It makes more sense for one to be able to specify which variable is reordered 
rather than have one variable always positioned as the source. This is the 
approach that we've taken with the ConditionalMutualInformationCalculator now

Original comment by joseph.lizier on 14 Jan 2013 at 11:41

GoogleCodeExporter commented 9 years ago
Need to make this change for the discrete.ConditionalMutualInformationCalculator

Original comment by joseph.lizier on 17 Oct 2013 at 6:39

GoogleCodeExporter commented 9 years ago
The Kraskov MI calculators currently shuffle the "destination" variable instead 
of the source (done by other MI calculators). This will still return an 
equivalent distribution (it's MI, not conditional MI in play here), but is 
inconsistent.

Original comment by joseph.lizier on 26 Mar 2014 at 5:08