-
Notifications
You must be signed in to change notification settings - Fork 179
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
feat(hardware-testing): Configurable plunger acceleration per liquid-class #13038
feat(hardware-testing): Configurable plunger acceleration per liquid-class #13038
Conversation
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.
looks good to me
Codecov Report
@@ Coverage Diff @@
## edge #13038 +/- ##
==========================================
- Coverage 72.49% 72.04% -0.45%
==========================================
Files 2376 1560 -816
Lines 65595 51223 -14372
Branches 7269 3257 -4012
==========================================
- Hits 47551 36904 -10647
+ Misses 16313 13809 -2504
+ Partials 1731 510 -1221
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
looks good
Overview
This PR adds plunger acceleration (ul/sec/sec) as one of the configurations that are applied during pipetting motions. This allows the system default accelerations to be used during homing and positioning, and then these faster accelerations can be used during only aspirate/dispense/blow-out movements.
Test Plan
Run 2x trials on the following (make sure they are wet-runs so liquid is actually moved):
Changelog
Review requests
Risk assessment