fix(api): peek pipette bug fixes #16985
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
On testing the peek pipette on a robot, there are two bugs:
em_pipette
, but the shared-data file getting loaded in was that of the regular 8 channel pipettesdefault_max_speed
was getting overwritten byot3api::set_system_constraints_for_plunger_acceleration
This pr contains fixes for both of these that have been tested on the robot.
Changelog
update_constraints_for_emulsifying_pipette
, instead just place a check for the high speed pipettes insideset_system_constraints_for_plunger_acceleration
EIGHT_CHANNEL_EM
types so the correct shared-data file gets loaded inTest Plan
cache_instruments
bind_sensor_output_requests
directed to the peek pipette