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

autotest: add test showing throttle satuation problem #27114

Merged

Conversation

peterbarker
Copy link
Contributor

No description provided.

@jschall
Copy link
Contributor

jschall commented May 21, 2024

Before #27106
image
After #27106
image

@peterbarker
Copy link
Contributor Author

I've now modified this to be a regression test, and I intend to merge this after CI has passed.

The test fails on master but passed on top of #27106

The test will be disabled - whatever fixes the problem should remove the line which disables the test.

autotest: verbose minimum_duration in wait_message_field_values
@peterbarker peterbarker force-pushed the pr/plane-tecs-throttle-test branch from 54c6230 to 8bee74b Compare May 21, 2024 03:26
@peterbarker peterbarker merged commit f930ba7 into ArduPilot:master May 21, 2024
60 checks passed
@peterbarker peterbarker deleted the pr/plane-tecs-throttle-test branch May 22, 2024 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants