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

Plane: Improved Failsafe Handling During Auto-Landings #12264

Closed
SUI-Eric opened this issue Sep 11, 2019 · 2 comments
Closed

Plane: Improved Failsafe Handling During Auto-Landings #12264

SUI-Eric opened this issue Sep 11, 2019 · 2 comments

Comments

@SUI-Eric
Copy link

Issue details

If an aircraft is performing an automatic landing (after a DO_LAND_START) and encounters a failsafe with an action of LAND, it will restart the landing sequence from the beginning. If - for example - an aircraft encounters a critical battery failsafe on final approach, the resulting aborted landing and go-around can be especially taxing on the already-low battery. Suggest that failsafe behavior be updated such that if the aircraft is already performing a landing sequence and encounters a failsafe with LAND action (or perhaps also RTL with RTL_AUTOLAND 1) that it simply continue the approach rather than revert to DO_LAND_START.

Version

AP3.9.8

Platform

[ ] All
[ ] AntennaTracker
[ ] Copter
[x] Plane
[ ] Rover
[ ] Submarine

Airframe type

Fixed-Wing Airplane

Hardware type

Cube Black

Logs

Can share some log excerpts upon request.

@tridge tridge self-assigned this Sep 11, 2019
@rmackay9 rmackay9 changed the title Improved Failsafe Handling During Auto-Landings Plane: Improved Failsafe Handling During Auto-Landings Sep 11, 2019
@MattKear
Copy link
Contributor

MattKear commented Dec 17, 2019

I have put in a PR. Hope to resolve this issue soon.

@IamPete1
Copy link
Member

IamPete1 commented Feb 18, 2020

closed by #13065 for basic missions this should now be fixed, if you have complex missions where there are jumps after the do land start you will need to enable distance based calculation with MIS_OPTIONS bit 1, ie + 2 to what you have set

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants