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

Port VTOL mission item resets and QC rework #21665

Merged
merged 2 commits into from Jun 2, 2023

Conversation

sfuhrer
Copy link
Contributor

@sfuhrer sfuhrer commented May 31, 2023

Port of #21641 and #21598 to 1.14

RomanBapst and others added 2 commits May 31, 2023 19:42
* refactored uncommanded descend quadchute
- use fixed altitude error threshold
- compute error relative to higest altitude the vehicle has achieved
since it has flown below the altitude reference (setpoint)

* disabled altitude loss quadchute by default

* altitude loss quadchute: added protection against invalid local z


---------

Signed-off-by: RomanBapst <bapstroman@gmail.com>
…_transition_item

set_vtol_transition_item sets the params of the mission item directly
to values that make sense for NAV_CMD_DO_VTOL_TRANSITION, but don't
for other NAV_CMDs. So make sure that whenever we use it, we then in
the next step reset the touched mission_item fields.

Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
@sfuhrer sfuhrer added the Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing! label May 31, 2023
@sfuhrer sfuhrer requested a review from RomanBapst May 31, 2023 17:45
@sfuhrer sfuhrer merged commit 487b9ca into release/1.14 Jun 2, 2023
3 of 4 checks passed
@sfuhrer sfuhrer deleted the pr-port-mission-item-reset-and-qc branch June 2, 2023 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants