-
Notifications
You must be signed in to change notification settings - Fork 178
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(shared-data): Re-apply update tip overlap values for Flex Pipettes (#15147) #15284
fix(shared-data): Re-apply update tip overlap values for Flex Pipettes (#15147) #15284
Conversation
6fa6494
to
cab20f6
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.
Approved previously in #15147, looks good if and when the versioning considerations are solved.
a32d587
to
be140b8
Compare
After conducting hardware testing, we've formulated updated tip overlap values for every tip size and pipette model. These adjustments apply to 1-channel, 8-channel, and 96-channel pipettes. These revised values aim to mitigate overpressure occurrences in ABR. [https://docs.google.com/spreadsheets/d/16dEmqulO4SIKU6E0aYTfVQEnZRTn3MwNF_WB8K1KmRM/edit#gid=0](url) - Added v2 tip overlap values for all flex pipette-tip combos
be140b8
to
f4c790e
Compare
Change API version 2.19 to use the new tip overlap values that should reduce overpressure errors. Closes EXEC-452
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.
All looks correct! Good to go!
@@ -12,6 +12,7 @@ | |||
ENGINE_CORE_API_VERSION: Final = APIVersion(2, 14) | |||
SET_OFFSET_RESTORED_API_VERSION: Final = APIVersion(2, 18) | |||
|
|||
|
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.
snuck in there
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.
Nice! 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.
tested 2.18,2.19, and json -lgtm
This is #15147 again but this time outside the context of an extremely imminent release.
While this code needs to be merged, we should carefully consider how we do so. We can't ship this until we can separate this behavior by API level so that customers who may have tuned in offsets based on the old incorrect tip offsets won't have their protocols invalidated - this data needs to be loaded by API level.
Closes EXEC-452