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.

Sign up for GitHub

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(api): validation casing for mixed tip use between single and eight channels #15259

Merged
merged 3 commits into from
May 23, 2024
Merged

fix(api): validation casing for mixed tip use between single and eight channels #15259

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dc155c0
validation casing for mixed tip use for 8 channels
CaseyBatten May 23, 2024
7d77052
linting fixes
CaseyBatten May 23, 2024
d01accc
unit tests added
CaseyBatten May 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions api/src/opentrons/protocol_engine/state/tips.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -267,6 +267,14 @@ def _validate_tip_cluster(
elif all(wells[well] == TipRackWellState.USED for well in tip_cluster):
return None
else:
# In the case of an 8ch pipette where a column has mixed state tips we may simply progress to the next column in our search
if (
nozzle_map is not None
and len(nozzle_map.full_instrument_map_store) == 8
):
return None

# In the case of a 96ch we can attempt to index in by singular rows and columns assuming that indexed direction is safe
# The tip cluster list is ordered: Each row from a column in order by columns
tip_cluster_final_column = []
for i in range(active_rows):
Expand Down
Loading