zhangfengcdt commented on issue #124: URL: https://github.com/apache/sedona-db/issues/124#issuecomment-3341928127
> This is a valid concern. `geo` crate already has [Haversine](https://docs.rs/geo/latest/geo/algorithm/line_measures/metric_spaces/constant.Haversine.html) and [Geodesic](https://docs.rs/geo/latest/geo/algorithm/line_measures/metric_spaces/static.Geodesic.html) for point <-> point distance calculation. > > Sedona-db's KNN join requires yet another patch to be merged into upstream: [wherobots/geo-index#5](https://github.com/wherobots/geo-index/pull/5). I have also suggested defining DistanceMetric more flexibly for that patch: [#53 (review)](https://github.com/apache/sedona-db/pull/53#pullrequestreview-3208589822). Our geo-index was already refactored to use the geo crate distance function. w.r.t. the refactoring to neighbors_geometry to be indices based parameters, I will try to implement it (maybe coexist with the current geometries approach) before merging to upstream. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
