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.

Sign up for GitHub

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

Jump to bottom

Copter: Add support for altitude offset input in AUTO mode #20390

Closed

mustafa-gokce wants to merge 2 commits into ArduPilot:master from mustafa-gokce:copter_tune_altitude_auto

Contributor

mustafa-gokce commented Mar 28, 2022

With this PR, if the TUNE parameter is set to 59, users can add offsets to the vehicle's altitude in centimeters within (TUNE_MIN, TUNE_MAX).

mustafa-gokce added 2 commits

March 28, 2022 12:55


          Copter: Add support for altitude offset input in AUTO mode

fdc4479


          AC_AttitudeControl: Add altitude tune offset

08109d6

Contributor Author

mustafa-gokce commented Mar 28, 2022 •

edited

Loading

A couple of thoughts and issues:

There is no restriction for minimum and maximum values. I think at least the minimum value should be restricted to avoid a controlled crash.
Vehicle reaches the waypoints and goes to the next waypoints without an issue (Reached command #N appears on the screen) but not the last waypoint. In order to reach the last waypoint, the result altitude offset has to be set to 0.

hendjoshsr71 requested a review from rmackay9

April 12, 2022 01:01

hendjoshsr71 added the Copter label

mustafa-gokce closed this

mustafa-gokce deleted the copter_tune_altitude_auto branch

September 20, 2022 10:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment