Skip to content

Commit 9f9d268

Browse files
authored
feat(protocol-designer, step-generation): update mix for inPlace, expand tipstate for pipettes (#18512)
This PR wires up `aspirate/dispenseInPlace` commands to mix. It also expands tip state temporal properties for pipettes to include not only if a tip is attached, but what the tiprack URI of the attached tip is, so that we can check safe pipette movements in `moveToWell` commands through checking robot state at that command. In `mix` compound command creator, we now emit: - `moveToWell` - `aspirateInPlace` - `dispenseInPlace` rather than: - `aspirate` - `dispense` Also, I wire up pipette movement safety checks in `moveToWell`, requiring nozzle configuration to be passed. This has a large footprint, since I needed to update each implementation of `moveToWell` in step generation command creators.
1 parent fd0992b commit 9f9d268

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+1568
-1379
lines changed

app/src/organisms/ODD/QuickTransferFlow/__tests__/utils/quickTransferStepCommands.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,10 @@ const mockRobotState: TimelineFrame = {
9090
},
9191
},
9292
pipettes: {
93-
mockPipette: false,
93+
mockPipette: {
94+
hasTip: false,
95+
tiprackURI: null,
96+
},
9497
},
9598
},
9699
liquidState: {

components/src/organisms/ProtocolTimelineScrubber/PipetteVisuals.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export function PipetteMountViz(
6060
) : null}
6161
{pipetteEntity != null && pipetteId != null ? (
6262
<PipetteSideView
63-
allNozzlesHaveTips={timelineFrame.tipState.pipettes[pipetteId]}
63+
allNozzlesHaveTips={timelineFrame.tipState.pipettes[pipetteId].hasTip}
6464
allNozzleTipContents={Object.values(
6565
timelineFrame.liquidState.pipettes[pipetteId]
6666
)}

0 commit comments

Comments
 (0)