-
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
refactor(protocol-designer, step-generation): allow aluminum blocks t… #14174
Conversation
…o be moved, disallow adapters closes RQA-2060
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## edge #14174 +/- ##
=======================================
Coverage 70.40% 70.40%
=======================================
Files 2510 2510
Lines 71201 71202 +1
Branches 8971 8970 -1
=======================================
+ Hits 50127 50128 +1
Misses 18880 18880
Partials 2194 2194
Flags with carried forward coverage won't be shown. Click here to find out more.
|
const { dimensions, parameters } = labwareDef | ||
const { xDimension, yDimension } = dimensions | ||
|
||
const smallXDimension = xDimension < 127.75 |
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.
nit
maybe isSmallerXDimension
since it would be easy to understand that the const is boolean.
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.
oh ya good call! thanks!
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.
Followed the test plan and the sandbox worked as expected.
The changes look good to me.
…o be moved, disallow adapters
closes RQA-2060
Overview
The off-deck slideout now filters out adapters but keeps aluminum blocks. And aluminum blocks can not be moved only manually.
Test Plan
Create either a Flex or Ot-2 protocol. Add a gripper. On the design tab, click on the edit off-deck button and in the labware selection, the adapters should NOT be available. The aluminum blocks SHOULD be available. Click to add an aluminum block off-deck.
Create a move labware step and move the aluminum block off-deck and onto a slot. The step should be created correctly.
Create a new move labware step and move the aluminum block to a new slot (not off-deck) and make sure the
use gripper
checkbox is selected. This step should error and show a timeline error saying that the gripper cannot move aluminum blocks. If you edit the step and uncheck the the box, the step should work as expected.Changelog
LabwareSelectionModal
's util logic and filter out adapters if the slot is off-deckCANNOT_MOVE_WITH_GRIPPER
and add to i18nmoveLabware
atomic commandReview requests
see test plan
Risk assessment
low