Skip to content
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(shared-data): add custom & default thermocycler overlap offsets … #13750

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

jerader
Copy link
Collaborator

@jerader jerader commented Oct 10, 2023

…for labware (#13630)

  • added TC overlaps for nest & biorad pcr plates
  • added a backwards-compatible default overlap for TC labware on OT2
  • fix pd e2e migration fixture ---------

Overview

cherry picked this pr: #13630 to add to PD 7.0.0 release

Test Plan

TC testing Flex.json
testing TC OT-2.json
custom_labware_96_wells.json

See protocols + custom labware attached to this PR and please test on chore_release-7.0.1, the cherry-picked commit isn't in an alpha yet so just make sure you're up to date with that branch.

This test plan is copied over from the cherry picked PR:
The protocols place a nest PCR plate, a biorad plate and a custom pcr plate on the thermocycler one ofter the other to check for location precision.

Upload Flex protocol and check that:

  • when you run LPC, the pipette moves to accurate z-positions for the NEST and Biorad plates, and moves too high for the custom pcr plate

Upload OT-2 protocol and check that:

  • when you run LPC, the pipette moves to accurate z-positions for the NEST and Biorad plates, and an okay position for custom pcr plate. Also note that positional accuracy of the OT2 for labware on modules is not as high as the Flex so I wouldn't stress too much if there is an offset of a couple of mm.

Changelog

Review requests

Risk assessment

low, already has been tested on the release branch

…for labware (#13630)

* added TC overlaps for nest & biorad pcr plates
* added a backwards-compatible default overlap for TC labware on OT2
* fix pd e2e migration fixture
---------

Co-authored-by: Jethary <[email protected]>
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #13750 (b09c67c) into pd_release-7.0.0 (1f3cb50) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           pd_release-7.0.0   #13750   +/-   ##
=================================================
  Coverage             71.29%   71.29%           
=================================================
  Files                  2424     2424           
  Lines                 68203    68203           
  Branches               7975     7975           
=================================================
  Hits                  48622    48622           
  Misses                17711    17711           
  Partials               1870     1870           
Flag Coverage Δ
app 68.98% <ø> (ø)
g-code-testing 96.44% <ø> (ø)
labware-library 49.17% <ø> (ø)
notify-server 89.13% <ø> (ø)
protocol-designer 46.02% <ø> (ø)
shared-data 74.25% <ø> (ø)
step-generation 86.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
api/src/opentrons/protocol_engine/state/labware.py 100.00% <ø> (ø)

@jerader jerader marked this pull request as ready for review October 10, 2023 14:34
@jerader jerader requested review from a team as code owners October 10, 2023 14:34
@jerader jerader requested review from a team and removed request for a team October 10, 2023 14:34
Copy link
Member

@sanni-t sanni-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Backend changes look good 👍

Copy link
Contributor

@mjhuff mjhuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works as expected!

@jerader jerader merged commit 6c24a34 into pd_release-7.0.0 Oct 10, 2023
66 of 72 checks passed
@jerader jerader deleted the shared-data_labware-offsets-for-tc branch October 10, 2023 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants