From 2053ac95d36f52cc5c3e65c8878ea982640ac461 Mon Sep 17 00:00:00 2001 From: Seth Foster Date: Fri, 8 Dec 2023 16:26:15 -0500 Subject: [PATCH] fix(app): Remove probe check from module calibration (#14154) * fix(app): remove probe check from module cal Module calibration doesn't require a pipette id and thus none is available here (and in general I believe you can do it with a gripper) so this isn't really an appropriate check. * this was new too whoops probably should have been a sign --- .../ModuleWizardFlows/AttachProbe.tsx | 31 ++----------------- 1 file changed, 3 insertions(+), 28 deletions(-) diff --git a/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx b/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx index 2e05dedd3ec..ee9d0be66ba 100644 --- a/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx +++ b/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx @@ -22,7 +22,6 @@ import { import { Banner } from '../../atoms/Banner' import { StyledText } from '../../atoms/text' import { GenericWizardTile } from '../../molecules/GenericWizardTile' -import { ProbeNotAttached } from '../PipetteWizardFlows/ProbeNotAttached' import type { ModuleCalibrationWizardStepProps } from './types' interface AttachProbeProps extends ModuleCalibrationWizardStepProps { @@ -67,12 +66,8 @@ export const AttachProbe = (props: AttachProbeProps): JSX.Element | null => { 'module_wizard_flows', 'pipette_wizard_flows', ]) - const [showUnableToDetect, setShowUnableToDetect] = React.useState( - false - ) const moduleDisplayName = getModuleDisplayName(attachedModule.moduleModel) - const pipetteId = attachedPipette.serialNumber const attachedPipetteChannels = attachedPipette.data.channels let pipetteAttachProbeVideoSource, probeLocation @@ -159,12 +154,6 @@ export const AttachProbe = (props: AttachProbeProps): JSX.Element | null => { setErrorMessage('calibration adapter has not been loaded yet') return } - const verifyCommands: CreateCommand[] = [ - { - commandType: 'verifyTipPresence', - params: { pipetteId: pipetteId, expectedState: 'present' }, - }, - ] const homeCommands: CreateCommand[] = [ { commandType: 'home' as const, @@ -188,18 +177,12 @@ export const AttachProbe = (props: AttachProbeProps): JSX.Element | null => { }, ] - chainRunCommands?.(verifyCommands, false) + chainRunCommands?.(homeCommands, false) .then(() => { - chainRunCommands?.(homeCommands, false) - .then(() => { - proceed() - }) - .catch((e: Error) => { - setErrorMessage(`error starting module calibration: ${e.message}`) - }) + proceed() }) .catch((e: Error) => { - setShowUnableToDetect(true) + setErrorMessage(`error starting module calibration: ${e.message}`) }) } @@ -225,14 +208,6 @@ export const AttachProbe = (props: AttachProbeProps): JSX.Element | null => { )} ) - else if (showUnableToDetect) - return ( - - ) // TODO: add calibration loading screen and error screen else return (