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: Add rangefinder engagement distance parameter #29266

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rubenp02
Copy link
Contributor

@rubenp02 rubenp02 commented Feb 10, 2025

Introduced the RNGFND_LND_DIST parameter to specify the distance at which the rangefinder engages during landing. This allows the rangefinder to operate within a specified distance for landing purposes, while still measuring up to its maximum range for other applications.

When set to the default value of 0, the rangefinder engages as soon as it is within range, matching the existing behavior.

Key changes:

  • Added the RNGFND_LND_DIST parameter to control rangefinder engagement distance during landing.
  • Updated rangefinder logic to engage only when within the specified distance.
  • Changed the landing height getter to only apply the rangefinder correction and to return that the rangefinder is active if it's in use, instead of just in range.

Introduced the RNGFND_LND_DIST parameter to specify the distance at
which the rangefinder engages during landing. This allows the
rangefinder to operate within a specified distance for landing purposes,
while still measuring up to its maximum range for other applications.

When set to the default value of 0, the rangefinder engages as soon as
it is within range, matching the existing behavior.

Key changes:
- Added the RNGFND_LND_DIST parameter to control rangefinder engagement
  distance during landing.
- Updated rangefinder logic to engage only when within the specified
  distance.
- Changed the landing height getter to only apply the rangefinder
  correction and to return that the rangefinder is active if it's in
  use, instead of just in range.
@rubenp02 rubenp02 force-pushed the feature/add-param-rngfnd-lnd-dist branch from 8e7512c to 2a06ce2 Compare February 17, 2025 12:52
@rubenp02 rubenp02 marked this pull request as draft February 17, 2025 16:55
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