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

Charts: allow to hide elevation #1297

Open
Mugen36 opened this issue Jun 25, 2022 · 26 comments
Open

Charts: allow to hide elevation #1297

Mugen36 opened this issue Jun 25, 2022 · 26 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@Mugen36
Copy link

Mugen36 commented Jun 25, 2022

Is your feature request related to a problem? Please describe.
The elevation data on my tracks is completely non-sense. I am running 5 times around the block. And the graph shows that I climbed down 20 meters. And frankly, if one use Opentracks for evening runs, tracking the altitude is not that important anyways.

Describe the solution you'd like
Make recording elevation data optional. Implement an option that disables the recording of altitude data.

EDIT: As not recording altitude data is not feasible, a solution would be to give an option to not show altitude data in the graphs.

@Mugen36 Mugen36 added the enhancement New feature or request label Jun 25, 2022
@dennisguse
Copy link
Member

The absolute elevation from GPS or gain/loss from the barometric sensor?
And not showing it or not storing it or not exporting it?

Where is it shown that you don't want it?

@Mugen36
Copy link
Author

Mugen36 commented Jun 26, 2022

I was referring to the absolute elevation from the gps. I actually do not see any data from the barometric sensor.
The GPS-Beidoo-Glonass location data and the running speed seems fairly accurate. Accuracy usually <5m.
Yet the elevation data is completely garbage. Not only does it show a 20m climb-down where there is none. It is also roughly 40m too high.
I would prefer an option of not storing it, since it is just plainly false.

I see the elevation data in the graphs when I look at the specific tracks.

@Mugen36
Copy link
Author

Mugen36 commented Jun 26, 2022

At the core, I ultimately would blame Google for not putting a better GPS in their Pixels. Yet, since elevation data is usually far less accurate than horizontal location, I can imagine many more people have this issue.

@dennisguse
Copy link
Member

Well no.

  1. GPS accuracy for horizontal data is less accurate than vertical data (due to how GPS works)2
  2. exported data is in WGS84 as specified by KML/GPX.
    but this is not above ground aka sea level value. That's why OpenTracks converts for the UI the elevation into EGM2008.

And implementing this switch is too much effort; especially maintaining it.
The only thing would be drop data that is not accurate enough as reported by the GPS (horizontal accuracy is provided)

@Mugen36
Copy link
Author

Mugen36 commented Jun 27, 2022

And not showing it or not storing it or not exporting it?

Well, if not storing and not exporting, are not reasonable requests,
could I ask for an option for not showing elevation data in the graphs?

@dennisguse
Copy link
Member

That would work.
Interested to implement it?

@Mugen36
Copy link
Author

Mugen36 commented Jun 27, 2022

Sure

@Mugen36
Copy link
Author

Mugen36 commented Jun 27, 2022

May I ask, am I the only one who is getting nonsensical altitude values?

Or am I just the only one who is annoyed by seeing these in the graph?

@dennisguse
Copy link
Member

How would I know 😎
We have literally zero monitoring and know only stuff if somebody reports.
Can you share a screenshot of your chart?

@Mugen36
Copy link
Author

Mugen36 commented Jun 27, 2022

Here you go!

4 different days. The track is always the same. After the first half of my track, I turn around and run the same way back home.
I looked up the altitude of my track in Wikipedia. I am always running around 37m to 40m meters above sea level.
Screenshot_OpenTracks-01
Screenshot_OpenTracks-02
Screenshot_OpenTracks-03
Screenshot_OpenTracks-04

@dennisguse
Copy link
Member

Actually, we just removed the smoothing for the elevation (was only used in the charts) as it was broken #1241.

@Mugen36
Copy link
Author

Mugen36 commented Jun 28, 2022

I dont think smoothing would help much here. The base values are just too far off.
I did tests with different devices and in every case the altitude values were too unreliable to be of any use.
When you use opentracks, do you get any informative altitude data?

Please consider reopening this issue. Implementing it will be fairly easy. And I will definitely look into it although I am not familiar with the code base and quite pressed for time atm.

@dennisguse
Copy link
Member

@Mugen36 Just re-open it; the issue tracker is a place for everybody :)
The charts were actually configurable in MyTracks - so somewhere in the git history there is a commit for this. I guess I removed while trying to clean up as much as possible

PS/ I don't use the elevation data that much as I live in an area without any mountains.
So, for me it is not an issue that much.
And I don't use the charts for analytics.

@dennisguse dennisguse changed the title Make the recording of altitude data optional Charts: allow to hide elevation Jun 29, 2022
@Mugen36
Copy link
Author

Mugen36 commented Jun 29, 2022

@Mugen36 Just re-open it; the issue tracker is a place for everybody :)

I dont believe github allows me to reopen issue that were closed by a collaborator.

PS/ I don't use the elevation data that much as I live in an area without any mountains.

As do I, that is why I am annoyed to see bogus data, which I dont really need in the first place.

In any case, Thanks for your work on this project! I use the app everyday.

@dennisguse dennisguse reopened this Jun 29, 2022
@ghost
Copy link

ghost commented Jul 1, 2022

Regarding showing elevation data in general: Since we already have a way to customize recording layout where i can hide gain/loss data, would it be possible to customize track summary layout as well?

My Pixel 5 has a barometric sensor that apparently works in funky ways with OpenTracks. I've seen this issue here multiple times so i know it isn't in your power to change so there's no need to go through that in detail nor open another ticket about it, but i would simply prefer to not even see the gain/loss info on the track summary window. I realize this may sound like nitpicking, but i just did a 35km bike ride and having OpenTracks show me a 45 kilometer gain and 43 kilometer loss is just completely useless to even see there. I much prefer no data (or even slightly inaccurate data like calorie estimations in some apps) than completely false data like this. :)

@dennisguse
Copy link
Member

@crankc can you open a new issue for this?
Gain/loss is different than absolute elevation.
PS/ Can you check if #1307 helps with your pressure sensor?

@ghost
Copy link

ghost commented Jul 1, 2022

Sure, i opened a feature request regarding editing summary layouts and hiding unwanted data within.

About the pull request you linked, i'm not sure what to check? I'm not a developer, just an end user, but to my eyes it seems like it might help. I tested my phone with Trail Sense and sure enough, the barometric sensor values change rapidly every time i quickly move the phone, or even squeeze it gently. Now i know why it showed me 45 kilometers of gain after i had it in my pocket while cycling.

@dennisguse
Copy link
Member

@crankc if you signed into Github, you can download the APK for each pull request and install it manually (take a look at the Github Actions and then Nightly).
Not that convenient but it works.

@ghost
Copy link

ghost commented Jul 1, 2022

Thanks, i found it and commented in the relevant pull request thread #1307

@MartinH80
Copy link
Contributor

I recently started running again after several years. Happy to have found OpenTracks via fdroid!

My phone is a Pixel 6 and the altitude graphs are between 1 and 8 meters. That could be ok for where I live, but I can't match a the graph exactly with my observed reality.

More important to me is that I'm not interested in altitude and to me having two lines in one graph makes it harder to read exactly that what I'm looking for: my running pace.

So here another user who would be even more happy with OpenTracks when either (1) in settings the collection or visibility of this altitude data could be configured or (2) in the graphs themselves I could click on e.g. 'height' to hide this series.

I'm not a developer myself, but if I can do anything to help, please let me know!

@dennisguse
Copy link
Member

What about hiding the graph if less than 20m of difference was encountered?

@MartinH80
Copy link
Contributor

That solution would work for me! (but I have my doubts if this is a proper solution in general, 20m seems arbitrary and might not cover all use cases)

@hercek
Copy link
Contributor

hercek commented Dec 31, 2023

I vote for an option to hide altitude data in the charts. It would be useful for people who train mostly at the same altitude. For them, it clutters the charts with "garbage" since it is relatively imprecise.

@Brandon-Huu
Copy link

Why is this not an option yet?
I've running on a flat road and open tracks is telling me I climbed up a mountain both ways on my way to and from school

@dennisguse dennisguse added the help wanted Extra attention is needed label Jul 17, 2024
@dennisguse
Copy link
Member

@Brandon-Huu if you have time, you can work on this. Wasn't the highest priority...

@Brandon-Huu
Copy link

o7 I'll see what I can do

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants