-
Notifications
You must be signed in to change notification settings - Fork 300
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
Use sleep instead of post_delay in doNd #3576
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3576 +/- ##
=======================================
Coverage 65.69% 65.69%
=======================================
Files 225 225
Lines 30465 30465
=======================================
Hits 20014 20014
Misses 10451 10451 |
@jenshnielsen |
@FarBo Yes the sleep should be between set of setpoints and getting the parameter |
@FarBo I think this is close to being ready. Could you add a changelog entry? |
for more information, see https://pre-commit.ci
bors r+ |
This PR is created to change the delay time between setpoints in doNd functions. This is done because the current post_delay logic can unintentionally prolong the delay time if a ramped parameter is used, which the ramp time is shorter than the delay time provided in the doNd functions.
The PR is addressing this issue:
#3564