-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Support straight line mission landings #23576
Conversation
wing guidance - allows navigator to explicitly set the altitude acceptance radius - needed for staright line landing support * added ignore_alt_acceptance to position setpoint message to allow guidance logic to ignore altitude error on waypoint - can be useful to prevent loitering at a waypoint within a mission landing sequence Signed-off-by: RomanBapst <bapstroman@gmail.com> * Update src/modules/fw_pos_control/FixedwingPositionControl.cpp Co-authored-by: Alvaro Fernandez <alvaro@auterion.com> * fixed typo Signed-off-by: RomanBapst <bapstroman@gmail.com> --------- Signed-off-by: RomanBapst <bapstroman@gmail.com> Co-authored-by: Alvaro Fernandez <alvaro@auterion.com>
2b7bf96
to
f429111
Compare
I guess the small bump in altitude after accepting the last waypoint is due to the transition? |
Edit: ya makes sense... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would look into passing an "disregard altitude acceptance" flag from the groundstation to PX4 in cases where it should be ignored (e.g. during VTOL_LAND items). I don't think it should apply to every landing sequence.
Signed-off-by: RomanBapst <bapstroman@gmail.com>
… a mission landing for fixed wing vehicles Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
@KonradRudin @sfuhrer I added the fixed wing check as agreed. |
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
…position setpoint is larger 0 Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Solved Problem
When flying Fixed Wings in a space constrained area, it can be difficult to plan the usual orbit type mission landings.
There might simply not be enough space for the vehicle to orbit while it's descending in altitude.
Of course, one can just plan a mission with simple waypoints and add a VTOL LAND waypoint at the end. However, there is always the risk that the altitude difference between the waypoints is too big and the vehicle cannot follow the glide-slope, causing it to loiter at the next waypoint in order to reduce altitude further. This behavior is highly undesirable as part of a mission landing, as already explained above.
Fixes #{Github issue ID}
Solution
I propose a solution where all simple waypoints AFTER a land stark marker have their altitude acceptance radius set to INFINITY, which will cause the guidance logic to track altitude for these waypoints based on best effort.
Changelog Entry
For release notes:
Test coverage
Tested regularly in production but on a much older branch. Only SITL tested on main.
Context
Altitude profile for staright line landing. Notice, that for the first waypoints the altitude needs to reached, however, for the last waypoints (landing pattern), the altitude is not reached anymore, but the waypoint is still accepted.