-
-
Notifications
You must be signed in to change notification settings - Fork 461
fix(crsf): assign GPS Alt unique name rather than share Baro Alt #6995
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
Conversation
|
Welcome to the club. Yes, this is a pain especially if you are trying to use Alt in a calculated distance sensor. You are getting offered two Alt sensor without knowing which one is AGL Altitude (Baro Alt) or MSL Alt (GPS Alt). Good luck dude, the community will eventually succeed. I hope it's you. For context, there's most likely more: @JimB40's proposal Jan 2025: #5852
My proposal in Sep 2023: #4010
|
|
We also need to rename STR_SENSOR_GPSALT to STR_SENSOR_GALT for consistency |
|
STR_SENSOR_GPSALT is also used by other telemetry protocols, so it also needs to be renamed there, but it's a possibility. The other is to create a new name like STR_SENSOR_BALT for the baro altitude and display |
|
I'll take care of the renaming |
Why? |
|
Indeed. STR_SENSOR_GPS_ALT and STR_SENSOR_BARO_ALT make more sense IMO than STR_SENSOR_BALT and STR_SENSOR_GALT. |
Indeed, and quite silly they clashed in the first place given you can rename it... we know one is of type GPS_ID, and the other is of type BARO_ALT_ID, so why wouldn't default name wouldn't ensure to prevent clashes? Derp! Sorry I didn't see your last comment at the time otherwise I would have pushed that back up. |


With the release of Betaflight 2025.12, CRSF now transmits two different altitude values:
Currently, both of these values are displayed as "Alt" in EdgeTX, which makes it impossible to distinguish between them, even though they represent different data sources and have different characteristics (barometric altitude is in
cmordmwhile GPS altitude is only inm).With this PR, the telemetry data related to the GPS altitude is displayed as "GAlt" while the barometric altitude is displayed as "Alt"
Fixes #5852
Summary of changes:
Tested on a RM Pocket
