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

fix(step-generation): liquidState tracking for 96-channel column pickup #15076

Merged
merged 2 commits into from
May 2, 2024

Conversation

jerader
Copy link
Collaborator

@jerader jerader commented May 2, 2024

closes RESC-246 AUTH-373

Overview

This PR fixes a bug in the liquid state tracking specifically for using a 96-channel column partial tip configuration. The bug was that I forgot to consider partial tip for the 96-channel in the liquid state tracking all together so the protocol in the RESC ticket was tracking liquids as if all 96 nozzles were being used instead of 1 column

Test Plan

Upload the protocol attached to the RESC ticket on edge. The last transfer step should have a "not enough liquid in wells" warning. Upload the same protocol on this branch. The last transfer step should NOT have a "not enough liquid in wells" warning.

Changelog

  • update the dispense update liquid state component to grab the pipette's nozzle configuration. If nozzles is column then the channels is 8 instead of 96
  • fix the test
  • extend props to include the robot state and warnings and update all affected components

Review requests

see test plan

Risk assessment

low

@jerader jerader requested a review from a team as a code owner May 2, 2024 19:13
@jerader jerader requested review from koji, ncdiehl11 and a team and removed request for a team May 2, 2024 19:13
Copy link
Contributor

@koji koji left a comment

Choose a reason for hiding this comment

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

the sandbox worked as expected!

@jerader jerader merged commit 504bc57 into edge May 2, 2024
16 checks passed
@jerader jerader deleted the sg_liquid-state-update-96-channel branch May 2, 2024 19:57
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.

2 participants