fix hist generator

lineupjs · Dec 1, 2018 · 6955ccb · 6955ccb
1 parent 278ed00
commit 6955ccb
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/internal/math.ts b/src/internal/math.ts
@@ -1,5 +1,6 @@
 import {ICategory} from '../model';
 import {ISequence} from './interable';
+import {bisectLeft} from 'd3-array';
 
 export interface INumberBin {
   x0: number;
@@ -265,6 +266,7 @@ export function computeNormalizedStats(arr: ISequence<number>, numberOfBins?: nu
 
   const bin1 = 0 + binWidth;
   const binN = 1 - binWidth;
+  const binEnds = bins.map((d) => d.x1);
 
   const toBin = (v: number) => {
     if (v < bin1) {
@@ -273,7 +275,7 @@ export function computeNormalizedStats(arr: ISequence<number>, numberOfBins?: nu
     if (v >= binN) {
       return count - 1;
     }
-    return Math.max(Math.min(Math.round(v * (count - 1)), count - 1), 0);
+    return bisectLeft(binEnds, v);
   };
   return new LazyBoxPlotData(arr, {bins, toBin});
 }