-
Notifications
You must be signed in to change notification settings - Fork 4
CRSF Modes, Reset home #71
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
base: main
Are you sure you want to change the base?
Conversation
|
For test i am use XPlane 11 + INav HITL plugin + Inav + receivers ELRS and FrSky TW |
|
Great. I am, back home now and will double check all works my end then merge. |
|
@jaroszmm .. I have tested this with sport and looks like no issues. Can you please do a test on one of your elrs rigs - if all ok; we can merge. |
|
It should never show AIR. Airmode is not a flight mode. It is a modifier and applies to all flight modes. |
|
@r1000ru .. I have another PR up https://github.com/iNavFlight/ETHOS-Telemetry-Dashboard/actions/runs/18935398164 That includes the audio files for this. Maybe you could check all ok on your end? |
|
@MrD-RC ACRO or AIR - it is default flight mode value. If used other flight mode (including Air Modifier), we will never get a mention about Air. At the same time, the Air makes adjustments to flight control. I think it's worth mentioning this (especially since it's done on the OSD) |
Not at all. The LUA should match the OSD for consistency. AIR is never displayed in the INAV OSD. It hasn't been for years. As Airmode can apply just as much to Angle or Horizon as to Acro. It would be inconsistent behaviour to show AIR instead of the actual flight mode, which would be Acro. The telemetry code should not be sending AIR at all. It is likely a hang over from when the code was added, probably an import from BetaFlight. Which does incorrectly display AIR. |
|
Just noticed home reset is in modes too. This is not the right place to have this. It is not a flight mode. |
|
@MrD-RC , I am remove AIR mode. |
|
It may be worth checking the master INAV's
|
|
@MrD-RC , It is very good. I will add these modes |
|
It's worth keeping the work you already did. Then it will still work with INAV 8 and earlier. Just for information. The OpenTX telemetry script ignored AIR. it just showed ACRO if AIR was detected. |
Add CRSF telemetry flight modes: WAIT, ERROR, LAND, ANGLE HOLD, HOME RESET, ets
If HOME RESET is active - widget home point will be reseted to current coordinates.
Known issue: if used SPort-based telemetry and this flight modes, INav send value as in ACRO. It is not widget bug.