-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
MPC: LAND: Split Landing out of FlightTaskAuto into FlightTaskLand #23678
base: main
Are you sure you want to change the base?
Conversation
309d09b
to
e0e1a53
Compare
I wonder how does this effect PrecisionLand? The precland logic lives in navigator as a submode, and I believe puts the vehicle into FlightTaskAuto with a land setpoint type. Something to keep in mind! |
1b23655
to
d3d49fd
Compare
I now made it so that Landing Flightmode goes into FlightTaskLand, and Precision landing remains in FlightTaskAuto. I wasn't able to also integrate precision landing into it, as i dont have the means to test it, and FlightTaskLand is currently not set up to handle movement around to setpoint triplets as FlightTaskAuto. it just brakes, and lands. |
776b848
to
a1f1b9f
Compare
6a22182
to
f019e3a
Compare
1a853a8
to
46ef94b
Compare
46ef94b
to
f429026
Compare
6803b79
to
0ddbe61
Compare
0ddbe61
to
4d003fd
Compare
🔎 FLASH Analysispx4_fmu-v5x [Total VM Diff: 3560 byte (0.17 %)]
px4_fmu-v6x [Total VM Diff: 3560 byte (0.18 %)]
Updated: 2024-12-16T14:35:43 |
Solved Problem
The #23546 did not consider the case when we trigger land from Altitude or Manual FlightTask, where the commanded velocity can greatly exceed the velocity possible in FlightTaskAuto.
Solution
With this PR, we are splitting the Landing Logic into a separate FlighTask called FlightTaskLand, and handle the landing logic in there.
Test on Hardware
Changelog Entry
For release notes:
Dependent PR's
#23757 documents the part of temporarily extending the speed limits when already entering the mode too fast.
Test coverage