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

RTL: Add approaches for rtl to home and rally points #21049

Merged
merged 8 commits into from
Oct 31, 2023

Conversation

KonradRudin
Copy link
Contributor

@KonradRudin KonradRudin commented Feb 2, 2023

Solved Problem

When a VTOL performs a RTL to the home or rally position, it flies straight to the designated position and lands there. If it is too high, it will circle around the designated position. however, it is desired to actually define a landing pattern with a designated loiter position when performing a RTL for a vtol in Fixed wing mode.

Fixes #{Github issue ID}

Solution

  • Using the MAVLINK Safe point messages, we can not only upload safe point landing locations, but designated loiter positions as well. Multiple loiter points can be defined for the safe point. The storage order of the loiter points determine to which safe landing position the loiter point belongs to.
  • Update the RTL logic to check if there exists some loiter position for the rtl landing point. If it exists, choose the best approach best on current vehicle location and wind condition and the perform a mission landing approach.

Test coverage

Context

Related links, screenshot before/after, video

@RomanBapst RomanBapst force-pushed the rearrange_rtl_mission_mode branch from 4020dd0 to 8b8e0bf Compare March 10, 2023 12:06
@junwoo091400 junwoo091400 added the Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing! label Mar 24, 2023
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch from 8b8e0bf to 8f61320 Compare July 14, 2023 15:11
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch 3 times, most recently from b3e4f2a to b6c3a25 Compare August 7, 2023 07:17
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch from 5a0e5b6 to 511289b Compare August 8, 2023 12:56
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch 10 times, most recently from 9217ba0 to fd1187c Compare August 22, 2023 13:05
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch 3 times, most recently from cd74f2a to 495f139 Compare August 28, 2023 09:07
@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch 3 times, most recently from a3b7ff5 to 646c64b Compare August 28, 2023 15:28
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch from 495f139 to 5e2dcbd Compare August 29, 2023 07:20
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch from 5e2dcbd to 94b6c88 Compare September 6, 2023 11:13
@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch from ab82fda to a88c614 Compare September 28, 2023 07:36
@KonradRudin KonradRudin force-pushed the rearrange_rtl_mission_mode branch from abf33f7 to ca72fe7 Compare September 28, 2023 12:38
@sfuhrer sfuhrer force-pushed the rearrange_rtl_mission_mode branch from ca72fe7 to fa296e6 Compare September 29, 2023 09:13
Base automatically changed from rearrange_rtl_mission_mode to main September 29, 2023 12:25
@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch 2 times, most recently from 4e1c961 to e7ad374 Compare October 3, 2023 18:11
@KonradRudin KonradRudin marked this pull request as ready for review October 4, 2023 08:37
@KonradRudin
Copy link
Contributor Author

I put 2a5c60c and b898bf3 in a separate PR here since those are fixes not related to the rtl home approaches

@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch from 2596a2a to ff91f35 Compare October 18, 2023 06:36
@KonradRudin
Copy link
Contributor Author

rebased on main and squashed commits

@KonradRudin KonradRudin changed the title Add rtl vtol home approach RTL: Add approaches for rtl to home and rally points Oct 18, 2023
@KonradRudin
Copy link
Contributor Author

Added flight logs in the description

@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch from ff91f35 to e7ef59e Compare October 21, 2023 12:48
@KonradRudin KonradRudin force-pushed the add_rtl_vtol_home_approach branch from 021a33f to dcc5f67 Compare October 26, 2023 07:53
@RomanBapst RomanBapst merged commit 2e85037 into main Oct 31, 2023
86 checks passed
@RomanBapst RomanBapst deleted the add_rtl_vtol_home_approach branch October 31, 2023 13:16
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.

3 participants