Minor fix in values to histogram conversion

Filter out values outside the min and max boundaries
because they are assigned to uninitialized buckets
(outside min and max bounds).
This commit is contained in:
Sofia Papagiannaki 2019-03-09 13:38:57 +02:00 committed by Sofia Papagiannaki
parent 5ed3139d5a
commit 4cf698af10

View File

@ -43,6 +43,10 @@ export function convertValuesToHistogram(values: number[], bucketSize: number, m
} }
for (let i = 0; i < values.length; i++) { for (let i = 0; i < values.length; i++) {
// filter out values outside the min and max boundaries
if (values[i] < min || values[i] > max) {
continue;
}
const bound = getBucketBound(values[i], bucketSize); const bound = getBucketBound(values[i], bucketSize);
histogram[bound] = histogram[bound] + 1; histogram[bound] = histogram[bound] + 1;
} }