CI test build: crsf-gps-alt-msl (do not merge) - #3
CI test build: crsf-gps-alt-msl (do not merge)#3raphaelhunziker1202-stack wants to merge 2 commits into
Conversation
…s_alt_source) The altitude field of the CRSF GPS frame (0x02) so far carried INAV's estimated altitude relative to the arming point, while the CRSF specification intends this field to carry the GPS altitude (MSL), which is also what Betaflight sends and what the GAlt sensor on EdgeTX/OpenTX radios is expected to show. Add a new setting `crsf_gps_alt_source`: - ESTIMATED (default): estimated altitude above the arming point, identical to the previous behaviour - MSL: raw GNSS altitude above mean sea level The relative altitude remains available via the barometer/vario frame (Alt sensor) regardless of this setting. PG_TELEMETRY_CONFIG version bumped 8 -> 9.
The intermediate integer-centimeter step rounded negative estimated altitudes differently than the previous float expression (by up to 1 m). Use the original expression verbatim for the ESTIMATED path so the default behaviour is unchanged, and drop raw from the MSL description since gpsSol may carry the estimated fix when GPS fix estimation is enabled.
Branch Targeting SuggestionYou've targeted the
If This is an automated suggestion to help route contributions to the appropriate branch. |
|
RAM / Flash usage vs. base branch — commit
See RAM/flash optimization guide for techniques to reduce usage. |
1 participant
Draft PR only to trigger the CI firmware build on this fork for testing iNavFlight#11850. Do not merge.