natas_hw
/
jidt
mirror of https://github.com/jlizier/jidt
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Corrects the access at one point to the bins for the data points in the multivariate kernel estimator, which should use an IntArray object but did use an int[] object to access . Fixes #69.

This commit is contained in:
jlizier 2018-05-10 20:20:02 +10:00
parent 860d43dbca
commit 0e4455b2fd
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
java/source/infodynamics/measures/continuous/kernel/KernelEstimatorMultiVariate.java
View File

@ -596,6 +596,7 @@ public class KernelEstimatorMultiVariate implements Cloneable {
}
int multiDimBin = 0;
int[] multiDimBinArray = null;
IntArray intArrayMultiDimBin = null;
if (usingIntegerIndexBins) {
multiDimBin = getMultiDimBinIndex(observation);
count = observations[multiDimBin].size();
@ -612,7 +613,7 @@ public class KernelEstimatorMultiVariate implements Cloneable {
}
} else {
multiDimBinArray = getMultiDimBinArray(observation);
IntArray intArrayMultiDimBin = new IntArray(multiDimBinArray);
intArrayMultiDimBin = new IntArray(multiDimBinArray);
Vector<TimeStampedObservation> observationsInThisBin =
(Vector<TimeStampedObservation>) observationsByHash.get(intArrayMultiDimBin);
// Shouldn't need to check (observationsInThisBin != null) here since
@ -678,7 +679,7 @@ public class KernelEstimatorMultiVariate implements Cloneable {
}
} else {
Vector<TimeStampedObservation> observationsInThisBin =
(Vector<TimeStampedObservation>) observationsByHash.get(multiDimBinArray);
(Vector<TimeStampedObservation>) observationsByHash.get(intArrayMultiDimBin);
if (observationsInThisBin != null) {
for (int i = 0; i < observationsInThisBin.size(); i++) {
TimeStampedObservation tso = (TimeStampedObservation) observationsInThisBin.elementAt(i);