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: Limit possible climb/sink rate in FBWB/CRUISE to TECS limits #26856

Merged

Conversation

pieniacy
Copy link
Contributor

A simple constrain to never change target altitude more than configured TECS limits, even if FBWB_CLIMB_RATE is more than TECS_SINK_MAX or TECS_CLMB_MAX. Should not change anything for properly configured FBWB_CLIMB_RATE. This constrain is asymetrical, meaning climb or sink constrain may happen at different stick deflections if TECS_SINK_MAX is different than TECS_CLMB_MAX.

Tested with SITL with:

TECS_SINK_MAX 4
TECS_CLMB_MAX 6
FBWB_CLIMB_RATE 8

Before:
image

After:
image

@tridge
Copy link
Contributor

tridge commented Apr 22, 2024

@pieniacy nice, thanks!

@tridge tridge merged commit 4e29e1e into ArduPilot:master Apr 22, 2024
61 checks passed
@pieniacy pieniacy deleted the kp/constrain-fbwb-cruise-alt-to-tecs branch April 23, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants