-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Updated throttle behaviour during takeoff #6149
base: master
Are you sure you want to change the base?
Conversation
6567deb
to
b6a1864
Compare
@Hwurzburg I made an effort in updating the documentation of the takeoff throttle. |
@Georacer questions....what happens with option =1 but no airspeed sensor? that is not addressed here...also at one point you say max thr is imposed until lvl_alt,but also say it isnt for nav takeoffs higher up...whats correct? |
@Hwurzburg I have updated according to the now merged ArduPilot/ardupilot#27758.
In practice, if one does not have an airspeed sensor enabled, the logic will revert back to
With the new TECS PR, |
0c1308c
to
c7c086d
Compare
@Georacer I made some tweaks....I havent squashed everything yet so you could easily see the changes I made... |
@Georacer George can you resolve thee conflicts and squash and then I will merge, or do you want me to do that? |
To include the updates of ArduPilot/ardupilot#27174