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

[Feature] Sanity check for records #226

Open
sgofferj opened this issue Jul 30, 2022 · 4 comments
Open

[Feature] Sanity check for records #226

sgofferj opened this issue Jul 30, 2022 · 4 comments

Comments

@sgofferj
Copy link

I noticed that in the "records" on the my dashboard are some pretty unlikely values which probably come from GPS glitches. E.g. top speeds - 148.02 km/h for Cycling (transport) or 351.53 km/h for Walking.
How about some sanity checks for the records in the form of max. plausible speeds?

@SamR1
Copy link
Owner

SamR1 commented Aug 3, 2022

Hi,

gpxpy excludes top values to remove some GPS errors (and sometimes some correct values too, see #48)

Are these values related to workouts recently uploaded?

@sgofferj
Copy link
Author

sgofferj commented Aug 4, 2022

04.07.2022 most recent as far as I can tell. While I was inside a shop, the GPS jump some 100m.
Screenshot_20220804_105202
I can send the GPX if it helps.

@SamR1
Copy link
Owner

SamR1 commented Aug 27, 2022

Hi,

Sorry for the delay.

It seems there are some invalid values left after gpxpy processing. I have sometimes the same issue when I forget to stop/pause the recording (in this case, I edit the file manually to remove some invalid points and let gpxpy removing the remaining ones, but that's not the best solution...).

Adding max values will make the records less relevant. But it can be a workaround before reworking gpx file processing (and adding gpx data edition).
A additional solution can be to let user edit max speed value on FitTrackee, to overwrite value calculated by gpxpy.

@Firesphere
Copy link

@sgofferj You only got to 148km/h? Come on, put some effort in it!

Max. speed 730.77 km/h

On a serious note, I agree, I'd like to see these glitches solved as well. Other than that, this is a pretty cool application!

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

No branches or pull requests

3 participants