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

Navigator: Avoid attitude stepoint resets by not going into Takeoff mode if VTOL_Takeoff is selected in-air #22518

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

sfuhrer
Copy link
Contributor

@sfuhrer sfuhrer commented Dec 11, 2023

Solved Problem

Attitude resets happening when switching to Mission mode with VTOL_Takeoff selected in-air.

Chain:

  • Navigator sets the type to SETPOINT_TYPE_TAKEOFF for one sample
  • FW position controller goes into _control_mode_current= FW_POSCTRL_MODE_AUTO_TAKEOFF
  • FW position controller executes control_auto_takeoff() --> resets integrals here

Solution

Navigator: Avoid attitude stepoint resets by not going into Takeoff mode if VTOL_Takeoff is selected in-air

Changelog Entry

For release notes:

Bugfix: Navigator: Avoid attitude stepoint resets by not going into Takeoff mode if VTOL_Takeoff is selected in-air.

Test coverage

SITL

Context

Reproducible in SITL (start mission with VTOL_Takeoff, change mission in-air, re-upload, start mission again).

Without the fix:
image

With the fix:

image

Set the setpoint type to POSITION if already in air, not to TAKEOFF.

Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
@sfuhrer sfuhrer added the Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing! label Dec 11, 2023
@RomanBapst RomanBapst merged commit 8aab3e8 into main Dec 11, 2023
90 checks passed
@RomanBapst RomanBapst deleted the pr-navigator-fix-vtol-takeoff-type-in-air-main branch December 11, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing!
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants