KristofferC / NearestNeighbors.jl

High performance nearest neighbor data structures (KDTree and BallTree) and algorithms for Julia.
Other
413 stars 65 forks source link

Disable fast path update for minimum distance to hyper rectangle for `Chebyshev` since it doesn't work #192

Open KristofferC opened 2 months ago

KristofferC commented 2 months ago

It is unclear to me if this "optimization" can be done with Chebyshev but at least it gives the wrong values now (see the added test I made by just asserting that the optimized min computation gave the same result as the one from scratch) so disable it for now.

Have to benchmark that pulling this computation out into its own function didn't make things worse.