Skip to content

Conversation

@Allain18
Copy link

With the release of Betaflight 2025.12, CRSF now transmits two different altitude values:

  • GPS altitude in the GPS frame (0x02)
  • Barometric altitude (and vertical speed) in the new frame (0x09)

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 cm or dm while GPS altitude is only in m).

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:

  • Change ALT to GPSALT

Tested on a RM Pocket
20260115_134813

@mha1
Copy link
Contributor

mha1 commented Jan 15, 2026

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

image

My proposal in Sep 2023: #4010

image

@3djc
Copy link
Collaborator

3djc commented Jan 16, 2026

We also need to rename STR_SENSOR_GPSALT to STR_SENSOR_GALT for consistency

@Allain18
Copy link
Author

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 BAlt for example.

@3djc
Copy link
Collaborator

3djc commented Jan 16, 2026

I'll take care of the renaming

@philmoz
Copy link
Collaborator

philmoz commented Jan 17, 2026

We also need to rename STR_SENSOR_GPSALT to STR_SENSOR_GALT for consistency

Why?
The name of the string does not have to be the string value. IMO having a more meaningful string name is more useful.
Many of the other sensor string names are very different to the string value (having more understandable names).

@pfeerick
Copy link
Member

Indeed. STR_SENSOR_GPS_ALT and STR_SENSOR_BARO_ALT make more sense IMO than STR_SENSOR_BALT and STR_SENSOR_GALT.

@pfeerick
Copy link
Member

Yes, this is a pain

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.

@pfeerick pfeerick changed the title fix: crsf telemetry rename GPS Alt in display fix(crsf): assign GPS Alt unique name rather than share Baro Alt Jan 21, 2026
@pfeerick pfeerick added bug 🪲 Something isn't working telemetry 📶 labels Jan 21, 2026
@pfeerick pfeerick added this to the 2.12.0 milestone Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🪲 Something isn't working telemetry 📶

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CRSF Distinct GPS Altitude from Barometric Altiude

5 participants