Skip to content

Commit

Permalink
temp commit
Browse files Browse the repository at this point in the history
  • Loading branch information
jerader committed Jan 30, 2025
1 parent 42dc7bf commit 698bd4b
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 19 deletions.
5 changes: 0 additions & 5 deletions protocol-designer/src/steplist/substepTimeline.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,22 +66,17 @@ const _createNextTimelineFrame = (args: {
volume: args.volume,
activeTips: _getNewActiveTips(args.nextFrame.commands.slice(0, args.index)),
}
const command = args.command
const isAirGapCommand =
'meta' in command && command.meta != null && 'isAirGap' in command.meta

const newTimelineFrame =
args.command.commandType === 'aspirate' ||
args.command.commandType === 'aspirateInPlace'
? {
..._newTimelineFrameKeys,
source: args.wellInfo,
isAirGap: isAirGapCommand,
}
: {
..._newTimelineFrameKeys,
dest: args.wellInfo,
isAirGap: isAirGapCommand,
}
return newTimelineFrame
}
Expand Down
1 change: 0 additions & 1 deletion step-generation/src/commandCreators/atomic/aspirate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ import type { Point } from '../../utils'
export interface ExtendedAspirateParams extends AspDispAirgapParams {
tipRack: string
nozzles: NozzleConfigurationStyle | null
isAirGap?: boolean
}
/** Aspirate with given args. Requires tip. */
export const aspirate: CommandCreator<ExtendedAspirateParams> = (
Expand Down
1 change: 0 additions & 1 deletion step-generation/src/fixtures/data.ts
Original file line number Diff line number Diff line change
Expand Up @@ -104,4 +104,3 @@ export const DEST_LABWARE = 'destPlateId'
export const TROUGH_LABWARE = 'troughId'
export const DEFAULT_BLOWOUT_WELL = 'A1'
export const TIPRACK_1 = 'tiprack1Id'
export const AIR_GAP_META = { isAirGap: true } // to differentiate if the aspirate or dispense command is an air gap or not
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ export function forAspirate(
labwareDef,
params.wellName
)

console.log('volume in forAspirate', volume)
console.assert(
// @ts-expect-error (sa, 2021-05-03): this assert is unnecessary
uniq(wellsForTips).length === allWellsShared ? 1 : wellsForTips.length,
Expand Down
12 changes: 2 additions & 10 deletions step-generation/src/getNextRobotStateAndWarnings/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -63,19 +63,11 @@ function _getNextRobotStateAndWarningsSingleCommand(
assert(command, 'undefined command passed to getNextRobotStateAndWarning')
switch (command.commandType) {
case 'aspirate':
if (command.meta?.isAirGap === true) {
break
} else {
forAspirate(command.params, invariantContext, robotStateAndWarnings)
}
forAspirate(command.params, invariantContext, robotStateAndWarnings)
break

case 'dispense':
if (command.meta?.isAirGap === true) {
break
} else {
forDispense(command.params, invariantContext, robotStateAndWarnings)
}
forDispense(command.params, invariantContext, robotStateAndWarnings)
break

case 'blowout':
Expand Down
2 changes: 1 addition & 1 deletion step-generation/src/utils/misc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ export function splitLiquid(
sourceLiquidState: LocationLiquidState
): SourceAndDest {
const totalSourceVolume = getLocationTotalVolume(sourceLiquidState)

console.log('volume in splitLiquid', volume)
if (totalSourceVolume === 0) {
// Splitting from empty source
return {
Expand Down

0 comments on commit 698bd4b

Please sign in to comment.