Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for choosing the rate at the which the location is fetched based on distance. #74

Merged
merged 5 commits into from
Feb 23, 2021

Conversation

cybersokari
Copy link
Contributor

Added distanceFilter from distanceFilter in iOS.
Added distanceFilter from smallestDisplacement in Android
Added isMock from isFromMockProvider variable in Android, iOS is always false

@cybersokari cybersokari changed the title Added support for choosing the rate at the which the location is fetched based distance. Added support for choosing the rate at the which the location is fetched based on distance. Feb 19, 2021
@Almoullim Almoullim merged commit 93d4875 into Almoullim:master Feb 23, 2021
@aliciasteiman
Copy link

I'm using this plugin for an app for runners and being able to accurately track distance (in the background) is a huge component of the app. Right now I have the distance filter set to its default value 0, but I'm noticing that I'm collecting close to 2000 waypoints for a single run. Every time a new waypoint is added, I increase a variable "distance" by using the Geolocator.distanceBetween(waypoint, waypoint) function so that I can display dynamic distance tracking to the user as they run. However, this distance is wayyyy overestimated. I'm wondering if decreasing the number of waypoints will help, but it's a little unclear what distanceFilter represents. For example, if I set distanceFilter to 10 what does that mean?

@Almoullim
Copy link
Owner

@aliciasteiman

distanceFilter

iOS:

The minimum distance (measured in meters) a device must move horizontally before an update event is generated.

Android:

The minimum displacement between location updates in meters

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants