owntracks / booklet

The definitive OwnTracks documentation
http://owntracks.org/booklet/
43 stars 59 forks source link

locatorDisplace default value - code 500m vs documentation 0m #57

Closed Dopeyr closed 9 months ago

Dopeyr commented 4 years ago

Hi,

This is a question about the locatorDisplacement value. In the docs, it says:

locatorDisplacement: The smallest displacement in meters the user must move between location updates. Defaults to 0 and is an and relationship with interval. Can be used to only receive updates when the device has moved.

(bold is mine)

However, in the preferences the default value is 500

I think the documentation is confused with the default in the android LocationRequest docs, where 0 is the default, if you don't set it.

In my (limited) experience so far, 500m is the distance that you need to move to trigger an update.

So my question is: which is right? Should the default in the preferences be 0, or should the documents specify 500 as the default?

I'm not a Java dev, so I'm trying to piece things together "from the outside" - please let me know if I'm way off track :)

Thanks, Andy.

boogymantoo commented 4 years ago

Hi Andy,

The "tricky thing" about locatorDisplacement and locatorInterval is the fact that there is an and relationship between the two: you need to have moved (at least) locatorDisplacement meters in a timespan of locatorInterval seconds - else, no position update will be delivered.

I presume that the statement "Defaults to 0" in the documentation was indeed meant as "Defaults to 0 on the Android platform". Might be confusing indeed.

Oh, and I think you can state, by definition, that you are always on your own track(s).

Best regards, Chris (just a member here, not a contributor in the project).

Dopeyr commented 4 years ago

Oh, and I think you can state, by definition, that you are always on your own track(s).

😄

growse commented 3 years ago

I think we could do with a little clarity on the booklet. This is a confusing subject and can be non-intuitive. I'll migrate this over.