Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

LatLng#distanceTo with TurfMeasurement#distance #14220

Merged
merged 1 commit into from
Apr 1, 2019
Merged

Conversation

tobrun
Copy link
Member

@tobrun tobrun commented Mar 25, 2019

Closes #14216, now that we use Turf as part of the Mapbox Maps SDK for Android in #11447. I feel we should use Turf whenever possible (vs writing the goespatial math ourselves). Validated that this PR produces non-NaN values for the use-case described in #14216 and added a test for it.

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Mar 25, 2019
@tobrun tobrun added this to the release-m milestone Mar 25, 2019
@tobrun tobrun self-assigned this Mar 25, 2019
@tobrun tobrun requested a review from LukasPaczos March 25, 2019 12:46
@tobrun tobrun force-pushed the tvn-turf-distance-to branch 3 times, most recently from f659c47 to 8a4136d Compare March 26, 2019 11:45
@tobrun tobrun merged commit 3e106c3 into master Apr 1, 2019
@tobrun tobrun deleted the tvn-turf-distance-to branch April 1, 2019 14:11
@Cloud9Clone
Copy link

Hello! Is the issue with the 0.0 distance for close objects solved?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LatLng distanceTo returns null for very close points
3 participants