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

navigator : Waypoint acceptance radius usable for all vehicle type #23605

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sbtjagu
Copy link
Contributor

@sbtjagu sbtjagu commented Aug 23, 2024

Solved Problem

I wanted to make it possible to use the acceptance radius defined by a waypoint for the new ackermann module, and found that there was a condition in navigator module allowing it only for rotary wing vehicles.

Solution

  • Add a new parameter NAV_USE_WP_RAD
  • if parameter NAV_USE_WP_RAD is enabled, use the max between waypoint acceptance radius and NAV_ACC_RAD (NAV_ACC_RAD acting as the minimal acceptance radius usable)
  • NAV_USE_WP_RAD disabled by default, and enabled in airframes of rotary wing vehicles to make it works as before.

Changelog Entry

For release notes:

New parameter: NAV_USE_WP_RAD

Test coverage

  • SITL with x500 with NAV_USE_WP_RAD enabled and disabled

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.

1 participant