-
Notifications
You must be signed in to change notification settings - Fork 17.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
Fix heli ice/turbine cooldown and add autotest #28310
Fix heli ice/turbine cooldown and add autotest #28310
Conversation
dea3f25
to
d197221
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this will flap, despite the races.
self.set_parameter("RC6_OPTION", 161) | ||
self.set_parameter("H_RSC_RAMP_TIME", RAMP_TIME) | ||
self.set_parameter("H_RSC_SETPOINT", SETPOINT) | ||
self.set_parameter("H_RSC_IDLE", IDLE) | ||
self.set_parameter("H_RSC_CLDWN_TIME", COOLDOWN_TIME) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set_parameters
can be faster.
self.set_rc(6, 2000) | ||
self.wait_statustext('Turbine startup') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a race condition here, should collect statustext in the context and check_context=True
This PR fixes the ice/turbine cool down feature in Heli RSC.
This also adds an autotest to prevent future regression.