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

VSpd telemetry error accumulates in flight #9987

Open
dingrgh opened this issue Apr 28, 2024 · 7 comments
Open

VSpd telemetry error accumulates in flight #9987

dingrgh opened this issue Apr 28, 2024 · 7 comments

Comments

@dingrgh
Copy link

dingrgh commented Apr 28, 2024

Current Behavior

Platform: iNav FC mounted in a motor glider.
VSpd telemetry parameter accumulates an error with a offset of up to +2m/s after about 30 minutes in the air. At the start of the flight, there is zero error. At the end of the flight, with the aircraft sitting on the ground, telemetry shows it is still climbing. This error is also noticeable in flight if the vario tones are enabled (i.e. a climb indicated when the glider is actually descending).
Screenshot 2024-04-28 at 3 36 51 PM

Steps to Reproduce

  1. Arm. Launch and fly.
  2. After about, say, 30 minutes in the air, land.
  3. Check VSpd telemetry with the aircraft sitting on the ground to observe the offset.
  4. Confirm that the Alt telemetry figure is correct (i.e. not changing).

Expected behavior

At the end of the flight, the VSPD telemetry should show zero m/s when sitting stationary on the ground.

Suggested solution(s)

The error was probably introduced in code changes for barometric and GPS alt.

Additional context

This VSpd error only started after I updated from iNav 6.0 to iNav 7.1.0.
iNav Dump: (https://pastebin.com/7ZSPq3p6)


  • FC Board name and vendor: Matek F405-WSE
  • INAV version string: INAV/MATEKF405SE 7.1.0 Mar 27 2024 / 13:52:59 (59a6ee6)
@dingrgh
Copy link
Author

dingrgh commented May 2, 2024

I did more flights with iNav 7.1.0 to confirm that the problem was repeatable. I even tried setting CLI parameter inav_w_z_gps_v to zero (default is 0.1) to remove all dependence on GPS vertical speed but it did not fix the problem.

I downgraded the firmware to iNav 7.0.0 and can confirm that the problem went away. I can only conclude that some change to the Vertical Speed code from 7.0.0 to 7.1.0 is causing the problem.

Summary: Rolling back to iNav 7.0.0 is a workaround for this issue.

@Frank0587
Copy link

Hey,
I had similar problems. The Baro was no longer used for altitude and vario. The default setting for “inav_use_gps_no_baro” has been changed from OFF to ON. After the CLI command “set inav_use_gps_no_baro = OFF” it works perfectly again. I hope this will solve your problem too.

@dingrgh
Copy link
Author

dingrgh commented May 14, 2024

Hey, I had similar problems. The Baro was no longer used for altitude and vario. The default setting for “inav_use_gps_no_baro” has been changed from OFF to ON. After the CLI command “set inav_use_gps_no_baro = OFF” it works perfectly again. I hope this will solve your problem too.

I will try this and report back in a week or so...

@dingrgh
Copy link
Author

dingrgh commented May 15, 2024

The VSpd issue may be related to #10040.

@sensei-hacker
Copy link
Collaborator

sensei-hacker commented May 15, 2024

The VSpd issue may be related to #10040.

I created #100400 based partly on reading this and finding out that inav_use_gps_no_baro doesn't do what some thought it did. It does exactly what the name says - it uses the GPS and not the baro.

Suggest turning off inav_use_gps_no_baro.

@dingrgh
Copy link
Author

dingrgh commented May 17, 2024

I have done several flights (iNav 7.1.1) with “inav_use_gps_no_baro” set to OFF. Unfortunately, the VSpd residual offset problem is still present.

So the problem is not likely to be related to #10040 but is a result of something that was changed in the Z velocity position estimator code between iNav 7.0.0 and iNav 7.1.x.

To be clear, there is no problem observed with the altitude telemetry. Only the VSpd telemetry.

@dingrgh
Copy link
Author

dingrgh commented May 26, 2024

Update: Workaround found...

Version: iNav 7.1.1

Things I tried that didn't solve the problem:

  • Setting "inav_use_gps_no_baro" to OFF.
  • Setting "inav_w_z_gps_v" to zero (default was 0.1) in an attempt to remove all GPS vertical speed weighting
  • Setting "inav_w_z_baro_p" to 2.0 (default was 0.35) in an attempt to increase weighting of baro altitude.

What worked finally:

  • I kept the above three changes and also changed "inav_w_z_gps_p" to zero (default was 0.2) to remove all GPS altitude weight. The VSpd telemetry now shows zero after landing (other than the expected small blips plus or minus 0.1m/s due to baro noise) while sitting stationary on the ground.

As mentioned in a previous post, rolling back to iNav 7.0.0 (without the changes above) also works around the problem.

Clearly, the problem introduced by code changes in iNav 7.1.0 will have to be fixed as I suspect it will affect other navigation functions in some annoying, hard-to-pinpoint way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants