node-binarysearch

node-binarysearch copied to clipboard

binarysearch.closest with decimals

1

```js const index = binary.closest([25, 33, 50], 32.99) expect(index).to.eq(1) // this throws and the index is 0. I would expect it to be 1 // does the compare closest function...

Lee182

`closest` does too many comparisons

4

I haven't closely examined the reason why, but in some very basic testing, there were more comparisons than there should have been for an efficient binary search of the `closest`...

rocketraman