Skip to content

[Backport 1.16] modules/navigator: Fix position setpoint update logic in Mission RTL#25861

Merged
sfuhrer merged 4 commits intoPX4:release/1.16from
msberk:fix_rtl_sp_logic_116bp
Nov 3, 2025
Merged

[Backport 1.16] modules/navigator: Fix position setpoint update logic in Mission RTL#25861
sfuhrer merged 4 commits intoPX4:release/1.16from
msberk:fix_rtl_sp_logic_116bp

Conversation

@msberk
Copy link
Contributor

@msberk msberk commented Nov 3, 2025

backport #25600

issue at: #25633

@sfuhrer

msberk and others added 4 commits November 3, 2025 11:47
Currently, when proceeding to the landing point the previous setpoint is not updated, which results in an unexpected and off course landing pattern in fixed wing. (see PX4#25436)
@msberk msberk changed the title [Backport 1.16] Fix position setpoint update logic in Mission RTL [Backport 1.16] modules/navigator: Fix position setpoint update logic in Mission RTL Nov 3, 2025
@sfuhrer sfuhrer merged commit 6b5cfcc into PX4:release/1.16 Nov 3, 2025
62 checks passed
Spir0u pushed a commit to aithon-robotics/PX4-Autopilot that referenced this pull request Jan 26, 2026
…previous waypoint correctly(PX4#25861)

This aligns setActiveMissionItems() in rtl_direct_mission_land.cpp and in rtl_mission_fast.cpp with what was already in mission.cpp. It probably was on oversight when the RTL restructure happened. The FW landing requires the previous waypoint to be correctly set, that's why it was only noticeable there.

* Fix position setpoint update logic in Mission RTL

Currently, when proceeding to the landing point the previous setpoint is not updated, which results in an unexpected and off course landing pattern in fixed wing. (see PX4#25436)

* Change to work more like `mission.cpp`

* Fix rtl_direct_misssion_land formatting for style guide

* rtl_mission_fast: fix FW landing by setting previous wp in landing

Signed-off-by: Silvan <[email protected]>

---------

Signed-off-by: Silvan <[email protected]>
Co-authored-by: Silvan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants