-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Ardupilot: Add Baro Alt Offset Compensation #11538
Conversation
Is this ready for review/merge @HTRamsey ? |
let me do the one fix mentioned in my edit real quick, and rebase to fix the linux build issue. Edit: Okay, when that builds then it is ready. |
e3fea8c
to
2ea7063
Compare
Is this okay to merge? |
You might want to tag someone from the Ardupilot team here. |
@peterbarker had checked it out previously via discord |
I would say then it's good to go. |
2ea7063
to
14964d5
Compare
The last commit currently seems to bring unrelated changes though... maybe you want to double check? |
I rebased onto master to make sure builds still pass |
Uses available pressure & temperature sensors to adjust altitude offset on drones for long flights using the params BARO_GND_TEMP & BARO_ALT_OFFSET. No current method of enabling, but the logic is there.