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

Threshold for jump detection is too low in some cases #106

Closed
ray66 opened this issue Mar 4, 2024 · 2 comments
Closed

Threshold for jump detection is too low in some cases #106

ray66 opened this issue Mar 4, 2024 · 2 comments
Labels
component: core Activity management, explorer tiles type: enhancement New feature or request
Milestone

Comments

@ray66
Copy link
Contributor

ray66 commented Mar 4, 2024

When the time difference between two track points is greater than 30 seconds, the connection between the two points is not considered as part of the activity. No line is drawn on the map and tiles between the two points are not added to the explorer tile map.

While this seems ok for sports-oriented activities like road cycling, the 30 seconds threshold is too low in many other use cases. Stops for taking pictures frequently exceed 30 seconds but should not be considered as activity breaks. Some devices have an option like "store location on significant change in direction only" which results in no points stored while moving on a straight line. This also may result in time difference between two points exceeding 30 seconds.

In order to prevent such breaks of activities, the time threshold should be either increased or made configurable.

Another solution would be to take into account the distance between the two points.

@martin-ueding
Copy link
Owner

With the changes from #90 I have also seen this in my own activities:

visualization

And my Garmin watch is set such that it continuously records, without stopping for breaks. I'll have to look into it. But it indeed seems that the threshold is too low.

@martin-ueding martin-ueding added the type: enhancement New feature or request label Mar 4, 2024
@martin-ueding
Copy link
Owner

The straight line thing is hard to do properly in order to prevent both false-positives and false-negatives.

I've now added an additional criterion that one must have a break longer than 30 s and also a jump of more than 100 m before it is considered a new segment. That should fix the issue with taking pictures, for instances.

It is not automatically applied to all existing activities, one would have to re-scan them.

I'll close this for now, but feel free to reopen when it doesn't solve your problem with version 0.21.0.

@martin-ueding martin-ueding added this to the 0.21.0 milestone Mar 22, 2024
@martin-ueding martin-ueding added the component: core Activity management, explorer tiles label Jul 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: core Activity management, explorer tiles type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants