kdtree icon indicating copy to clipboard operation
kdtree copied to clipboard

`KDNode.dist` returns squared distance while `search_nn_dist` requires euclidean distance

Open MichaelSuen-thePointer opened this issue 2 years ago • 1 comments

The two lines below, one requires dist to be euclidean distance, and the other requires it to be squared distance: https://github.com/stefankoegl/kdtree/blob/master/kdtree.py#L501 https://github.com/stefankoegl/kdtree/blob/master/kdtree.py#L508

And the best argument is not used https://github.com/stefankoegl/kdtree/blob/master/kdtree.py#L517

MichaelSuen-thePointer avatar May 18 '23 05:05 MichaelSuen-thePointer

Fixed 6 years ago in #43... Only a matter of time before It's merged

Ar-Kareem avatar Dec 02 '24 08:12 Ar-Kareem