opentrons 8.4.0a7__tar.gz → 8.4.0a9__tar.gz

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: opentrons
-Version: 8.4.0a7
+Version: 8.4.0a9
 Summary: The Opentrons API is a simple framework designed to make writing automated biology lab protocols easy.
 Author: Opentrons
 Author-email: engineering@opentrons.com
@@ -21,7 +21,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Topic :: Scientific/Engineering
 Requires-Python: >=3.10
 License-File: ../LICENSE
-Requires-Dist: opentrons-shared-data==8.4.0a7
+Requires-Dist: opentrons-shared-data==8.4.0a9
 Requires-Dist: aionotify==0.3.1
 Requires-Dist: anyio<4.0.0,>=3.6.1
 Requires-Dist: jsonschema<4.18.0,>=3.0.1
@@ -35,9 +35,9 @@ Requires-Dist: pyusb==1.2.1
 Requires-Dist: importlib-metadata>=1.0; python_version < "3.8"
 Requires-Dist: packaging>=21.0
 Provides-Extra: ot2-hardware
-Requires-Dist: opentrons-hardware==8.4.0a7; extra == "ot2-hardware"
+Requires-Dist: opentrons-hardware==8.4.0a9; extra == "ot2-hardware"
 Provides-Extra: flex-hardware
-Requires-Dist: opentrons-hardware[FLEX]==8.4.0a7; extra == "flex-hardware"
+Requires-Dist: opentrons-hardware[FLEX]==8.4.0a9; extra == "flex-hardware"
 
 .. _Full API Documentation: http://docs.opentrons.com
 

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_api/core/engine/instrument.py

@@ -834,7 +834,7 @@ class InstrumentCore(AbstractInstrument[WellCore, LabwareCore]):
             )
         )
 
-    def resin_tip_unseal(self, location: Location, well_core: WellCore) -> None:
+    def resin_tip_unseal(self, location: Location | None, well_core: WellCore) -> None:
         well_name = well_core.get_name()
         labware_id = well_core.labware_id
 
@@ -1188,9 +1188,9 @@ class InstrumentCore(AbstractInstrument[WellCore, LabwareCore]):
             cmd.GetNextTipParams(
                 pipetteId=self._pipette_id,
                 labwareIds=[tip_rack.labware_id for tip_rack in valid_tip_racks],
-                startingTipWell=starting_well.get_name()
-                if starting_well is not None
-                else None,
+                startingTipWell=(
+                    starting_well.get_name() if starting_well is not None else None
+                ),
             )
         )
         next_tip_info = result.nextTipInfo

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_api/core/instrument.py

@@ -206,7 +206,7 @@ class AbstractInstrument(ABC, Generic[WellCoreType, LabwareCoreType]):
     @abstractmethod
     def resin_tip_unseal(
         self,
-        location: types.Location,
+        location: types.Location | None,
         well_core: WellCoreType,
     ) -> None:
         ...

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_api/core/legacy/legacy_instrument_core.py

@@ -332,7 +332,7 @@ class LegacyInstrumentCore(AbstractInstrument[LegacyWellCore, LegacyLabwareCore]
 
     def resin_tip_unseal(
         self,
-        location: types.Location,
+        location: types.Location | None,
         well_core: WellCore,
     ) -> None:
         raise APIVersionError(api_element="Unsealing resin tips.")

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_api/core/legacy_simulator/legacy_instrument_core.py

@@ -298,7 +298,7 @@ class LegacyInstrumentCoreSimulator(
 
     def resin_tip_unseal(
         self,
-        location: types.Location,
+        location: types.Location | None,
         well_core: WellCore,
     ) -> None:
         raise APIVersionError(api_element="Unsealing resin tips.")

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_api/instrument_context.py

@@ -606,7 +606,20 @@ class InstrumentContext(publisher.CommandPublisher):
                     "Blow_out being performed on a tiprack. "
                     "Please re-check your code"
                 )
-            move_to_location = target.location or target.well.top()
+            if target.location:
+                # because the lower levels of blowout don't handle LiquidHandlingWellLocation and
+                # there is no "operation_volume" for blowout we need to convert the relative location
+                # given with a .meniscus to an absolute point. To maintain the meniscus behavior
+                # we can just add the offset to the current liquid height.
+                if target.location.meniscus_tracking:
+                    move_to_location = target.well.bottom(
+                        target.well.current_liquid_height()  # type: ignore [arg-type]
+                        + target.location.point.z
+                    )
+                else:
+                    move_to_location = target.location
+            else:
+                move_to_location = target.well.top()
             well = target.well
         elif isinstance(target, validation.PointTarget):
             move_to_location = target.location
@@ -1611,9 +1624,9 @@ class InstrumentContext(publisher.CommandPublisher):
                     (types.Location(types.Point(), labware=rack), rack._core)
                     for rack in transfer_args.tip_racks
                 ],
-                starting_tip=self.starting_tip._core
-                if self.starting_tip is not None
-                else None,
+                starting_tip=(
+                    self.starting_tip._core if self.starting_tip is not None else None
+                ),
                 trash_location=transfer_args.trash_location,
                 return_tip=return_tip,
             )
@@ -1727,9 +1740,9 @@ class InstrumentContext(publisher.CommandPublisher):
                     (types.Location(types.Point(), labware=rack), rack._core)
                     for rack in transfer_args.tip_racks
                 ],
-                starting_tip=self.starting_tip._core
-                if self.starting_tip is not None
-                else None,
+                starting_tip=(
+                    self.starting_tip._core if self.starting_tip is not None else None
+                ),
                 trash_location=transfer_args.trash_location,
                 return_tip=return_tip,
             )
@@ -1843,9 +1856,9 @@ class InstrumentContext(publisher.CommandPublisher):
                     (types.Location(types.Point(), labware=rack), rack._core)
                     for rack in transfer_args.tip_racks
                 ],
-                starting_tip=self.starting_tip._core
-                if self.starting_tip is not None
-                else None,
+                starting_tip=(
+                    self.starting_tip._core if self.starting_tip is not None else None
+                ),
                 trash_location=transfer_args.trash_location,
                 return_tip=return_tip,
             )
@@ -2019,7 +2032,7 @@ class InstrumentContext(publisher.CommandPublisher):
                 location=well,
             ),
         ):
-            self._core.resin_tip_unseal(location=well.top(), well_core=well._core)
+            self._core.resin_tip_unseal(location=None, well_core=well._core)
 
         return self
 

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/commands/liquid_probe.py

@@ -188,6 +188,12 @@ async def _execute_common(  # noqa: C901
             well_name=well_name,
             well_location=params.wellLocation,
         )
+        state_view.geometry.validate_probed_height(
+            labware_id=labware_id,
+            well_name=well_name,
+            pipette_id=pipette_id,
+            probed_height=z_pos,
+        )
     except PipetteLiquidNotFoundError as exception:
         move_result.state_update.set_pipette_ready_to_aspirate(
             pipette_id=pipette_id, ready_to_aspirate=True

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/commands/pipetting_common.py

@@ -180,7 +180,11 @@ async def prepare_for_aspirate(
     else:
         return SuccessData(
             public=EmptyResult(),
-            state_update=StateUpdate().set_fluid_empty(pipette_id=pipette_id),
+            state_update=StateUpdate()
+            .set_fluid_empty(pipette_id=pipette_id)
+            .set_pipette_ready_to_aspirate(
+                pipette_id=pipette_id, ready_to_aspirate=True
+            ),
         )
 
 

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/commands/prepare_to_aspirate.py

@@ -95,12 +95,7 @@ class PrepareToAspirateImplementation(
         if isinstance(prepare_result, DefinedErrorData):
             return prepare_result
         else:
-            return SuccessData(
-                public=PrepareToAspirateResult(),
-                state_update=prepare_result.state_update.set_pipette_ready_to_aspirate(
-                    pipette_id=params.pipetteId, ready_to_aspirate=True
-                ),
-            )
+            return self._transform_result(prepare_result)
 
 
 class PrepareToAspirate(

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/commands/seal_pipette_to_tip.py

@@ -1,12 +1,15 @@
 """Seal tips to pipette command request, result, and implementation models."""
 
 from __future__ import annotations
-from pydantic import Field, BaseModel
 from typing import TYPE_CHECKING, Optional, Type, Union
-from opentrons.types import MountType
-from opentrons.protocol_engine.types import MotorAxis
+
 from typing_extensions import Literal
+from pydantic import Field, BaseModel
+
+from opentrons_shared_data.errors.exceptions import PositionUnknownError
 
+from opentrons.types import MountType
+from opentrons.protocol_engine.types import MotorAxis
 from ..resources import ModelUtils, ensure_ot3_hardware
 from ..types import PickUpTipWellLocation, FluidKind, AspiratedFluid
 from .pipetting_common import (
@@ -27,7 +30,6 @@ from .command import (
 
 from opentrons.hardware_control import HardwareControlAPI
 from opentrons.hardware_control.types import Axis
-from ..state.update_types import StateUpdate
 
 if TYPE_CHECKING:
     from ..state.state import StateView
@@ -138,28 +140,50 @@ class SealPipetteToTipImplementation(
         """A relative press-fit pick up command using gantry moves."""
         prep_distance = tip_pick_up_params.prepDistance
         press_distance = tip_pick_up_params.pressDistance
-        retract_distance = -1 * (prep_distance + press_distance)
+        retract_distance = -1 * (press_distance) / 2
 
         mount_axis = MotorAxis.LEFT_Z if mount == MountType.LEFT else MotorAxis.RIGHT_Z
-
+        ot3_hardware_api = ensure_ot3_hardware(self._hardware_api)
         # TODO chb, 2025-01-29): Factor out the movement constants and relocate this logic into the hardware controller
-        await self._gantry_mover.move_axes(
-            axis_map={mount_axis: prep_distance}, speed=10, relative_move=True
+        try:
+            await self._gantry_mover.move_axes(
+                axis_map={mount_axis: prep_distance},
+                speed=10,
+                relative_move=True,
+                expect_stalls=True,
+            )
+        except PositionUnknownError:
+            # if this happens it's from the get position after the move and we can ignore it
+            pass
+
+        await ot3_hardware_api.update_axis_position_estimations(
+            self._gantry_mover.motor_axes_to_present_hardware_axes([mount_axis])
         )
 
         # Drive mount down for press-fit
-        await self._gantry_mover.move_axes(
-            axis_map={mount_axis: press_distance},
-            speed=10.0,
-            relative_move=True,
-            expect_stalls=True,
-        )
-        # retract cam : 11.05
-        await self._gantry_mover.move_axes(
-            axis_map={mount_axis: retract_distance}, speed=5.5, relative_move=True
+        try:
+            await self._gantry_mover.move_axes(
+                axis_map={mount_axis: press_distance},
+                speed=10.0,
+                relative_move=True,
+                expect_stalls=True,
+            )
+        except PositionUnknownError:
+            # if this happens it's from the get position after the move and we can ignore it
+            pass
+
+        await ot3_hardware_api.update_axis_position_estimations(
+            self._gantry_mover.motor_axes_to_present_hardware_axes([mount_axis])
         )
 
-        ot3_hardware_api = ensure_ot3_hardware(self._hardware_api)
+        try:
+            await self._gantry_mover.move_axes(
+                axis_map={mount_axis: retract_distance}, speed=5.5, relative_move=True
+            )
+        except PositionUnknownError:
+            # if this happens it's from the get position after the move and we can ignore it
+            pass
+
         await ot3_hardware_api.update_axis_position_estimations(
             self._gantry_mover.motor_axes_to_present_hardware_axes([mount_axis])
         )
@@ -283,13 +307,10 @@ class SealPipetteToTipImplementation(
             if hw_instr is not None:
                 hw_instr.set_current_volume(_SAFE_TOP_VOLUME)
 
-        state_update = StateUpdate()
-        state_update.update_pipette_tip_state(
+        state_update = move_result.state_update.update_pipette_tip_state(
             pipette_id=pipette_id,
             tip_geometry=tip_geometry,
-        )
-
-        state_update.set_fluid_aspirated(
+        ).set_fluid_aspirated(
             pipette_id=pipette_id,
             fluid=AspiratedFluid(kind=FluidKind.LIQUID, volume=_SAFE_TOP_VOLUME),
         )

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/commands/unseal_pipette_from_tip.py

@@ -3,12 +3,10 @@
 from __future__ import annotations
 
 from pydantic import Field
-from typing import TYPE_CHECKING, Optional, Type
+from typing import TYPE_CHECKING, Optional, Type, Final
 from typing_extensions import Literal
 
 from opentrons.protocol_engine.resources.model_utils import ModelUtils
-from opentrons.protocol_engine.types import MotorAxis
-from opentrons.types import MountType
 
 from ..types import DropTipWellLocation
 from .pipetting_common import (
@@ -56,6 +54,8 @@ _ExecuteReturn = (
     SuccessData[UnsealPipetteFromTipResult] | DefinedErrorData[StallOrCollisionError]
 )
 
+CUSTOM_TIP_LENGTH_MARGIN: Final = 10
+
 
 class UnsealPipetteFromTipImplementation(
     AbstractCommandImpl[UnsealPipetteFromTipParams, _ExecuteReturn]
@@ -85,6 +85,10 @@ class UnsealPipetteFromTipImplementation(
 
         well_location = params.wellLocation
 
+        tip_geometry = self._state_view.geometry.get_nominal_tip_geometry(
+            pipette_id, labware_id, well_name
+        )
+
         is_partially_configured = self._state_view.pipettes.get_is_partially_configured(
             pipette_id=pipette_id
         )
@@ -93,6 +97,7 @@ class UnsealPipetteFromTipImplementation(
             labware_id=labware_id,
             well_location=well_location,
             partially_configured=is_partially_configured,
+            override_default_offset=-(tip_geometry.length - CUSTOM_TIP_LENGTH_MARGIN),
         )
 
         move_result = await move_to_well(
@@ -106,14 +111,6 @@ class UnsealPipetteFromTipImplementation(
         if isinstance(move_result, DefinedErrorData):
             return move_result
 
-        # Move to an appropriate position
-        mount = self._state_view.pipettes.get_mount(pipette_id)
-
-        mount_axis = MotorAxis.LEFT_Z if mount == MountType.LEFT else MotorAxis.RIGHT_Z
-        await self._gantry_mover.move_axes(
-            axis_map={mount_axis: -14}, speed=10, relative_move=True
-        )
-
         await self._tip_handler.drop_tip(
             pipette_id=pipette_id,
             home_after=None,

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/errors/__init__.py

@@ -88,6 +88,7 @@ from .exceptions import (
     OffsetLocationInvalidError,
     FlexStackerLabwarePoolNotYetDefinedError,
     FlexStackerNotLogicallyEmptyError,
+    InvalidLabwarePositionError,
 )
 
 from .error_occurrence import ErrorOccurrence, ProtocolCommandFailedError
@@ -171,6 +172,7 @@ __all__ = [
     "OffsetLocationInvalidError",
     "FlexStackerLabwarePoolNotYetDefinedError",
     "FlexStackerNotLogicallyEmptyError",
+    "InvalidLabwarePositionError",
     # error occurrence models
     "ErrorOccurrence",
     "CommandNotAllowedError",

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/errors/exceptions.py

@@ -1281,3 +1281,15 @@ class FlexStackerLabwarePoolNotYetDefinedError(ProtocolEngineError):
         wrapping: Optional[Sequence[EnumeratedError]] = None,
     ) -> None:
         super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
+
+
+class InvalidLabwarePositionError(ProtocolEngineError):
+    """Raised when a labware position is internally invalid."""
+
+    def __init__(
+        self,
+        message: Optional[str] = None,
+        details: Optional[dict[str, Any]] = None,
+        wrapping: Optional[Sequence[EnumeratedError]] = None,
+    ) -> None:
+        super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)

{opentrons-8.4.0a7 → opentrons-8.4.0a9}/src/opentrons/protocol_engine/execution/pipetting.py

@@ -326,7 +326,7 @@ class HardwarePipettingHandler(PipettingHandler):
         well_name: str,
         well_location: WellLocation,
     ) -> LiquidTrackingType:
-        """Detect liquid level."""
+        """Return liquid level relative to the bottom of the well."""
         hw_pipette = self._state_view.pipettes.get_hardware_pipette(
             pipette_id=pipette_id,
             attached_pipettes=self._hardware_api.attached_instruments,