-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
#3250 breaks Blackbox GPS altitude #3500
Labels
Comments
stronnag
changed the title
#3250 breaks Blackbox altitude
#3250 breaks Blackbox GPS altitude
Jul 2, 2018
Fixed by #3501 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
#3250 naively changed the recording of altitude such that it could be negative. While we need to be able to record -ve altitudes, this is obviously NOT the correct solution, as it causes the recorded altitude to oscillate between +ve and -ve values, as the image below shows.
This affect has been observed on multiple vehicles, so its not just a stupid GPS. Code needs to be updated to use
blackboxWriteSignedVB
also.Fix under test.
The text was updated successfully, but these errors were encountered: