You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
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
Changed title: Alter MutualInfoCalculatorMultiVariate significance calculations to specify which variable should be reordered
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
Original issue reported on code.google.com by
joseph.lizier
on 6 Aug 2012 at 6:38The text was updated successfully, but these errors were encountered: