AP_Scripting: added copter_loiter_brake script #26971
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I haven't checked the script much for errors but it might be good to put terrain_brake in the name of the script. Loiter already has a brake included with it and we also have brake mode. So perhaps copter-terrain-brake or copter-loit-terr-brake. By the way, there have been requests over the year to enhance loiter mode so that it can do surface tracking using the terrain database. Maybe that's some thing we should add once Matt Kear's surface distance library is ready. |
IamPete1
reviewed
May 4, 2024
|
@rmackay9 renamed |
peterbarker
reviewed
May 7, 2024
rmackay9
approved these changes
May 7, 2024
for automatic BRAKE mode when flying in LOITER in steep terrain
MichelleRos
reviewed
May 7, 2024
| end | ||
|
|
||
| --[[ | ||
| main update function, called at 1Hz |
There was a problem hiding this comment.
Should this be 10 Hz instead, since update is 100ms?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
for automatic BRAKE mode when flying in LOITER in steep terrain