Skip to content

Plane: prevent learning bad ARSPD_RATIO #29455

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

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Mar 6, 2025

this prevents learning of ARSPD_RATIO when not in fly-forward or not armed

the scenario that happened on a real aircraft was gps glitches in a hanger combined with airspeed offset causing the learning of an ARSPD_RATIO of 4.0, resulting in a stall in an AUTO mission and a crash

this prevents learning of ARSPD_RATIO when not in fly-forward or not
armed

the scenario that happened on a real aircraft was gps glitches in a
hanger combined with airspeed offset causing the learning of an
ARSPD_RATIO of 4.0, resulting in a stall in an AUTO mission and a
crash
Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tridge tridge merged commit 1eb6491 into ArduPilot:master Mar 7, 2025
77 of 78 checks passed
@github-project-automation github-project-automation bot moved this to Pending in 4.6 Backports Mar 7, 2025
@tridge tridge mentioned this pull request Mar 7, 2025
94 tasks
@rmackay9 rmackay9 moved this from Pending to merge conflict in 4.6 Backports Mar 31, 2025
@rmackay9 rmackay9 mentioned this pull request Mar 31, 2025
6 tasks
@rmackay9
Copy link
Contributor

rmackay9 commented Apr 1, 2025

this is included in 4.6.0-beta5 via PR rmackay9#39

@rmackay9 rmackay9 moved this from merge conflict to 4.6.0-beta5 in 4.6 Backports Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 4.6.0-beta5
Development

Successfully merging this pull request may close these issues.

3 participants