opentrons 8.7.0a6__py3-none-any.whl → 8.7.0a7__py3-none-any.whl

opentrons/protocol_engine/commands/thermocycler/set_target_lid_temperature.py

@@ -1,26 +1,36 @@
 """Command models to start heating a Thermocycler's lid."""
 from __future__ import annotations
-from typing import Optional, TYPE_CHECKING
+from typing import Optional, TYPE_CHECKING, Any
 from typing_extensions import Literal, Type
 
 from pydantic import BaseModel, Field
+from pydantic.json_schema import SkipJsonSchema
 
 from ..command import AbstractCommandImpl, BaseCommand, BaseCommandCreate, SuccessData
 from ...errors.error_occurrence import ErrorOccurrence
 
 if TYPE_CHECKING:
     from opentrons.protocol_engine.state.state import StateView
-    from opentrons.protocol_engine.execution import EquipmentHandler
+    from opentrons.protocol_engine.execution import EquipmentHandler, TaskHandler
 
 
 SetTargetLidTemperatureCommandType = Literal["thermocycler/setTargetLidTemperature"]
 
 
+def _remove_default(s: dict[str, Any]) -> None:
+    s.pop("default", None)
+
+
 class SetTargetLidTemperatureParams(BaseModel):
     """Input parameters to set a Thermocycler's target lid temperature."""
 
     moduleId: str = Field(..., description="Unique ID of the Thermocycler Module.")
     celsius: float = Field(..., description="Target temperature in °C.")
+    taskId: str | SkipJsonSchema[None] = Field(
+        None,
+        description="Id for the background task that manages the temperature.",
+        json_schema_extra=_remove_default,
+    )
 
 
 class SetTargetLidTemperatureResult(BaseModel):
@@ -30,6 +40,11 @@ class SetTargetLidTemperatureResult(BaseModel):
         ...,
         description="The target lid temperature that was set after validation.",
     )
+    taskId: str | SkipJsonSchema[None] = Field(
+        None,
+        description="The task id for the setTargetBlockTemperature",
+        json_schema_extra=_remove_default,
+    )
 
 
 class SetTargetLidTemperatureImpl(
@@ -43,10 +58,12 @@ class SetTargetLidTemperatureImpl(
         self,
         state_view: StateView,
         equipment: EquipmentHandler,
+        task_handler: TaskHandler,
         **unused_dependencies: object,
     ) -> None:
         self._state_view = state_view
         self._equipment = equipment
+        self._task_handler = task_handler
 
     async def execute(
         self,
@@ -63,12 +80,19 @@ class SetTargetLidTemperatureImpl(
             thermocycler_state.module_id
         )
 
-        if thermocycler_hardware is not None:
-            await thermocycler_hardware.set_target_lid_temperature(target_temperature)
+        async def set_target_lid_temperature(task_handler: TaskHandler) -> None:
+            if thermocycler_hardware is not None:
+                await thermocycler_hardware.set_target_lid_temperature(
+                    target_temperature
+                )
+                await thermocycler_hardware.wait_for_lid_target()
 
+        task = await self._task_handler.create_task(
+            task_function=set_target_lid_temperature, id=params.taskId
+        )
         return SuccessData(
             public=SetTargetLidTemperatureResult(
-                targetLidTemperature=target_temperature
+                targetLidTemperature=target_temperature, taskId=task.id
             ),
         )
 

opentrons/protocol_engine/commands/thermocycler/start_run_extended_profile.py

@@ -0,0 +1,191 @@
+"""StartRunProfile command request, result, and implementation models."""
+
+from __future__ import annotations
+from pydantic import BaseModel, Field
+from typing import List, Optional, TYPE_CHECKING, overload, Union, Any
+from typing_extensions import Literal, Type
+from pydantic.json_schema import SkipJsonSchema
+
+from opentrons.hardware_control.modules.types import ThermocyclerStep, ThermocyclerCycle
+
+from ..command import AbstractCommandImpl, BaseCommand, BaseCommandCreate, SuccessData
+from ...errors.error_occurrence import ErrorOccurrence
+from .run_extended_profile import ProfileStep, ProfileCycle
+
+if TYPE_CHECKING:
+    from opentrons.protocol_engine.state.state import StateView
+    from opentrons.protocol_engine.execution import (
+        TaskHandler,
+        EquipmentHandler,
+    )
+    from opentrons.protocol_engine.state.module_substates.thermocycler_module_substate import (
+        ThermocyclerModuleSubState,
+    )
+
+StartRunExtendedProfileCommandType = Literal["thermocycler/startRunExtendedProfile"]
+
+
+def _remove_default(s: dict[str, Any]) -> None:
+    s.pop("default", None)
+
+
+class StartRunExtendedProfileStepParams(BaseModel):
+    """Input parameters for an individual Thermocycler profile step."""
+
+    celsius: float = Field(..., description="Target temperature in °C.")
+    holdSeconds: float = Field(
+        ..., description="Time to hold target temperature at in seconds."
+    )
+    rampRate: float | SkipJsonSchema[None] = Field(
+        None,
+        description="How quickly to change temperature in °C/second.",
+        json_schema_extra=_remove_default,
+    )
+
+
+class StartRunExtendedProfileParams(BaseModel):
+    """Input parameters to run a Thermocycler profile."""
+
+    moduleId: str = Field(..., description="Unique ID of the Thermocycler.")
+    profileElements: List[Union[ProfileStep, ProfileCycle]] = Field(
+        ...,
+        description="Elements of the profile. Each can be either a step or a cycle.",
+    )
+    blockMaxVolumeUl: float | SkipJsonSchema[None] = Field(
+        None,
+        description="Amount of liquid in uL of the most-full well"
+        " in labware loaded onto the thermocycler.",
+        json_schema_extra=_remove_default,
+    )
+    taskId: str | None = Field(None, description="The id of the profile task")
+
+
+class StartRunExtendedProfileResult(BaseModel):
+    """Result data from running a Thermocycler profile."""
+
+    taskId: str = Field(..., description="The id of the profile task")
+
+
+def _transform_profile_step(
+    step: ProfileStep, thermocycler_state: ThermocyclerModuleSubState
+) -> ThermocyclerStep:
+    return ThermocyclerStep(
+        temperature=thermocycler_state.validate_target_block_temperature(step.celsius),
+        hold_time_seconds=step.holdSeconds,
+        ramp_rate=thermocycler_state.validate_ramp_rate(step.rampRate, step.celsius),
+    )
+
+
+@overload
+def _transform_profile_element(
+    element: ProfileStep, thermocycler_state: ThermocyclerModuleSubState
+) -> ThermocyclerStep:
+    ...
+
+
+@overload
+def _transform_profile_element(
+    element: ProfileCycle, thermocycler_state: ThermocyclerModuleSubState
+) -> ThermocyclerCycle:
+    ...
+
+
+def _transform_profile_element(
+    element: Union[ProfileStep, ProfileCycle],
+    thermocycler_state: ThermocyclerModuleSubState,
+) -> Union[ThermocyclerStep, ThermocyclerCycle]:
+    if isinstance(element, ProfileStep):
+        return _transform_profile_step(element, thermocycler_state)
+    else:
+        return ThermocyclerCycle(
+            steps=[
+                _transform_profile_step(step, thermocycler_state)
+                for step in element.steps
+            ],
+            repetitions=element.repetitions,
+        )
+
+
+class StartRunExtendedProfileImpl(
+    AbstractCommandImpl[
+        StartRunExtendedProfileParams, SuccessData[StartRunExtendedProfileResult]
+    ]
+):
+    """Execution implementation of a Thermocycler's run profile command."""
+
+    def __init__(
+        self,
+        state_view: StateView,
+        equipment: EquipmentHandler,
+        task_handler: TaskHandler,
+        **unused_dependencies: object,
+    ) -> None:
+        self._state_view = state_view
+        self._equipment = equipment
+        self._task_handler = task_handler
+
+    async def execute(
+        self, params: StartRunExtendedProfileParams
+    ) -> SuccessData[StartRunExtendedProfileResult]:
+        """Run a Thermocycler profile."""
+        thermocycler_state = self._state_view.modules.get_thermocycler_module_substate(
+            params.moduleId
+        )
+        thermocycler_hardware = self._equipment.get_module_hardware_api(
+            thermocycler_state.module_id
+        )
+
+        profile = [
+            _transform_profile_element(element, thermocycler_state)
+            for element in params.profileElements
+        ]
+        target_volume: Optional[float]
+        if params.blockMaxVolumeUl is not None:
+            target_volume = thermocycler_state.validate_max_block_volume(
+                params.blockMaxVolumeUl
+            )
+        else:
+            target_volume = None
+
+        async def start_run_profile(task_handler: TaskHandler) -> None:
+            if thermocycler_hardware is not None:
+                async with task_handler.synchronize_cancel_latest(
+                    thermocycler_state.module_id
+                ):
+                    await thermocycler_hardware.execute_profile(
+                        profile=profile, volume=target_volume
+                    )
+
+        task = await self._task_handler.create_task(
+            task_function=start_run_profile, id=params.taskId
+        )
+
+        return SuccessData(
+            public=StartRunExtendedProfileResult(taskId=task.id),
+        )
+
+
+class StartRunExtendedProfile(
+    BaseCommand[
+        StartRunExtendedProfileParams, StartRunExtendedProfileResult, ErrorOccurrence
+    ]
+):
+    """A command to start the execution of a Thermocycler profile run without waiting for its completion."""
+
+    commandType: StartRunExtendedProfileCommandType = (
+        "thermocycler/startRunExtendedProfile"
+    )
+    params: StartRunExtendedProfileParams
+
+    _ImplementationCls: Type[StartRunExtendedProfileImpl] = StartRunExtendedProfileImpl
+
+
+class StartRunExtendedProfileCreate(BaseCommandCreate[StartRunExtendedProfileParams]):
+    """A request to execute a Thermocycler profile run."""
+
+    commandType: StartRunExtendedProfileCommandType = (
+        "thermocycler/startRunExtendedProfile"
+    )
+    params: StartRunExtendedProfileParams
+
+    _CommandCls: Type[StartRunExtendedProfile] = StartRunExtendedProfile

opentrons/protocol_engine/commands/touch_tip.py

@@ -114,7 +114,7 @@ class TouchTipImplementation(
 
         if self._state_view.labware.get_has_quirk(labware_id, "touchTipDisabled"):
             raise TouchTipDisabledError(
-                f"Touch tip not allowed on labware {labware_id}"
+                f"Touch tip not allowed on labware {self._state_view.labware.get_display_name(labware_id)}"
             )
 
         if self._state_view.labware.is_tiprack(labware_id):

opentrons/protocol_engine/commands/unsafe/unsafe_place_labware.py

@@ -14,12 +14,11 @@ from opentrons.protocol_engine.errors.exceptions import (
     CannotPerformGripperAction,
     GripperNotAttachedError,
 )
-from opentrons.types import Point
-
 from ...types import (
     DeckSlotLocation,
     ModuleModel,
     OnDeckLabwareLocation,
+    GripperMoveType,
 )
 from ..command import AbstractCommandImpl, BaseCommand, BaseCommandCreate, SuccessData
 from ...errors.error_occurrence import ErrorOccurrence
@@ -100,22 +99,6 @@ class UnsafePlaceLabwareImplementation(
             LabwareUri(params.labwareURI)
         )
 
-        # todo(mm, 2024-11-06): This is only correct in the special case of an
-        # absorbance reader lid. Its definition currently puts the offsets for *itself*
-        # in the property that's normally meant for offsets for its *children.*
-        final_offsets = self._state_view.labware.get_child_gripper_offsets(
-            labware_definition=definition, slot_name=None
-        )
-        drop_offset = (
-            Point(
-                final_offsets.dropOffset.x,
-                final_offsets.dropOffset.y,
-                final_offsets.dropOffset.z,
-            )
-            if final_offsets
-            else None
-        )
-
         if isinstance(params.location, DeckSlotLocation):
             self._state_view.addressable_areas.raise_if_area_not_in_deck_configuration(
                 params.location.slotName.id
@@ -139,7 +122,7 @@ class UnsafePlaceLabwareImplementation(
         await ot3api.update_axis_position_estimations([Axis.X, Axis.Y])
 
         # Place the labware down
-        await self._start_movement(ot3api, definition, location, drop_offset)
+        await self._start_movement(ot3api, definition, location)
 
         return SuccessData(public=UnsafePlaceLabwareResult())
 
@@ -148,7 +131,6 @@ class UnsafePlaceLabwareImplementation(
         ot3api: OT3HardwareControlAPI,
         labware_definition: LabwareDefinition,
         location: OnDeckLabwareLocation,
-        drop_offset: Optional[Point],
     ) -> None:
         gripper_homed_position = await ot3api.gantry_position(
             mount=OT3Mount.GRIPPER,
@@ -156,13 +138,15 @@ class UnsafePlaceLabwareImplementation(
         )
 
         to_labware_center = self._state_view.geometry.get_labware_grip_point(
-            labware_definition=labware_definition, location=location
+            labware_definition=labware_definition,
+            location=location,
+            move_type=GripperMoveType.DROP_LABWARE,
+            user_additional_offset=None,
         )
 
         movement_waypoints = get_gripper_labware_placement_waypoints(
             to_labware_center=to_labware_center,
             gripper_home_z=gripper_homed_position.z,
-            drop_offset=drop_offset,
         )
 
         # start movement

opentrons/protocol_engine/commands/wait_for_tasks.py

@@ -0,0 +1,98 @@
+"""WaitForTasks command request, result, and implementation models."""
+from __future__ import annotations
+from pydantic import BaseModel, Field
+from typing import Optional, Type, TYPE_CHECKING
+from typing_extensions import Literal
+
+from .command import AbstractCommandImpl, BaseCommand, BaseCommandCreate, SuccessData
+from ..errors.error_occurrence import ErrorOccurrence, ProtocolCommandFailedError
+from ..errors.exceptions import TaskFailedError
+
+if TYPE_CHECKING:
+    from ..execution import TaskHandler, RunControlHandler
+    from ..state.state import StateView
+
+
+WaitForTasksCommandType = Literal["waitForTasks"]
+
+
+class WaitForTasksParams(BaseModel):
+    """Payload required to annotate execution with a WaitForTasks."""
+
+    task_ids: list[str] = Field(
+        ...,
+        description="The list of task ids to wait for.",
+    )
+
+
+class WaitForTasksResult(BaseModel):
+    """Result data from the execution of a WaitForTasks command."""
+
+    task_ids: list[str] = Field(
+        ...,
+        description="The list of completed task ids.",
+    )
+
+
+class WaitForTasksImplementation(
+    AbstractCommandImpl[WaitForTasksParams, SuccessData[WaitForTasksResult]]
+):
+    """WaitForTasks command implementation."""
+
+    def __init__(
+        self,
+        task_handler: TaskHandler,
+        run_control: RunControlHandler,
+        state_view: StateView,
+        **kwargs: object,
+    ) -> None:
+        self._task_handler = task_handler
+        self._run_control = run_control
+        self._state_view = state_view
+
+    async def execute(
+        self, params: WaitForTasksParams
+    ) -> SuccessData[WaitForTasksResult]:
+        """Checks for existance of task id and then asynchronously waits for the valid, specified tasks to finish."""
+        # Raises the exception if we don't have a valid task id.
+        for task_id in params.task_ids:
+            _ = self._state_view.tasks.get(task_id)
+
+        await self._run_control.wait_for_tasks(params.task_ids)
+
+        failed_tasks = self._state_view.tasks.get_failed_tasks(params.task_ids)
+        if failed_tasks:
+            raise TaskFailedError(
+                message=f"{len(failed_tasks)} tasks failed.",
+                details={"failed_task_ids": failed_tasks},
+                wrapping=[
+                    ProtocolCommandFailedError(
+                        original_error=self._state_view.tasks.get_finished(
+                            task_id
+                        ).error
+                    )
+                    for task_id in failed_tasks
+                ],
+            )
+        return SuccessData(public=WaitForTasksResult(task_ids=params.task_ids))
+
+
+class WaitForTasks(
+    BaseCommand[WaitForTasksParams, WaitForTasksResult, ErrorOccurrence]
+):
+    """WaitForTasks command model."""
+
+    commandType: WaitForTasksCommandType = "waitForTasks"
+    params: WaitForTasksParams
+    result: Optional[WaitForTasksResult] = None
+
+    _ImplementationCls: Type[WaitForTasksImplementation] = WaitForTasksImplementation
+
+
+class WaitForTasksCreate(BaseCommandCreate[WaitForTasksParams]):
+    """WaitForTasks command request model."""
+
+    commandType: WaitForTasksCommandType = "waitForTasks"
+    params: WaitForTasksParams
+
+    _CommandCls: Type[WaitForTasks] = WaitForTasks

opentrons/protocol_engine/errors/__init__.py

@@ -57,6 +57,7 @@ from .exceptions import (
     InvalidTargetSpeedError,
     InvalidTargetTemperatureError,
     InvalidBlockVolumeError,
+    InvalidRampRateError,
     InvalidHoldTimeError,
     InvalidWavelengthError,
     CannotPerformModuleAction,
@@ -90,6 +91,7 @@ from .exceptions import (
     FlexStackerLabwarePoolNotYetDefinedError,
     FlexStackerNotLogicallyEmptyError,
     InvalidLabwarePositionError,
+    InvalidModuleOrientation,
 )
 
 from .error_occurrence import ErrorOccurrence, ProtocolCommandFailedError
@@ -151,6 +153,7 @@ __all__ = [
     "NoTargetTemperatureSetError",
     "InvalidTargetTemperatureError",
     "InvalidTargetSpeedError",
+    "InvalidRampRateError",
     "InvalidBlockVolumeError",
     "InvalidHoldTimeError",
     "InvalidLiquidError",
@@ -174,6 +177,7 @@ __all__ = [
     "FlexStackerLabwarePoolNotYetDefinedError",
     "FlexStackerNotLogicallyEmptyError",
     "InvalidLabwarePositionError",
+    "InvalidModuleOrientation",
     # error occurrence models
     "ErrorOccurrence",
     "CommandNotAllowedError",

opentrons/protocol_engine/errors/exceptions.py

@@ -413,6 +413,36 @@ class WellDoesNotExistError(ProtocolEngineError):
         super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
 
 
+class NoTaskFoundError(ProtocolEngineError):
+    """Raised when referencing a task that does not exist.
+
+    This error could be raised if a protocol references a task before it
+    has been created.
+    """
+
+    def __init__(
+        self,
+        message: Optional[str] = None,
+        details: Optional[Dict[str, Any]] = None,
+        wrapping: Optional[Sequence[EnumeratedError]] = None,
+    ) -> None:
+        """Build a NoTaskFoundError."""
+        super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
+
+
+class TaskFailedError(ProtocolEngineError):
+    """Raised when waiting on a task that failed."""
+
+    def __init__(
+        self,
+        message: Optional[str] = None,
+        details: Optional[Dict[str, Any]] = None,
+        wrapping: Optional[Sequence[EnumeratedError]] = None,
+    ) -> None:
+        """Build a TaskFailedError."""
+        super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
+
+
 class PipetteNotLoadedError(ProtocolEngineError):
     """Raised when referencing a pipette that has not been loaded."""
 
@@ -825,6 +855,19 @@ class InvalidTargetTemperatureError(ProtocolEngineError):
         super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
 
 
+class InvalidRampRateError(ProtocolEngineError):
+    """Raised when attempting to set an invalid ramp rate."""
+
+    def __init__(
+        self,
+        message: Optional[str] = None,
+        details: Optional[Dict[str, Any]] = None,
+        wrapping: Optional[Sequence[EnumeratedError]] = None,
+    ) -> None:
+        """Build a InvalidRampRateError."""
+        super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
+
+
 class InvalidBlockVolumeError(ProtocolEngineError):
     """Raised when attempting to set an invalid block max volume."""
 
@@ -1306,3 +1349,15 @@ class InvalidLabwarePositionError(ProtocolEngineError):
         wrapping: Optional[Sequence[EnumeratedError]] = None,
     ) -> None:
         super().__init__(ErrorCodes.GENERAL_ERROR, message, details, wrapping)
+
+
+class InvalidModuleOrientation(ProtocolEngineError):
+    """Raised when a module orientation is invalid for a slot id."""
+
+    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/protocol_engine/execution/__init__.py

@@ -21,6 +21,7 @@ from .run_control import RunControlHandler
 from .hardware_stopper import HardwareStopper
 from .door_watcher import DoorWatcher
 from .status_bar import StatusBarHandler
+from .task_handler import TaskHandler
 from ..resources.file_provider import FileProvider
 
 # .thermocycler_movement_flagger omitted from package's public interface.
@@ -46,5 +47,6 @@ __all__ = [
     "DoorWatcher",
     "RailLightsHandler",
     "StatusBarHandler",
+    "TaskHandler",
     "FileProvider",
 ]

opentrons/protocol_engine/execution/command_executor.py

@@ -35,6 +35,7 @@ from .tip_handler import TipHandler
 from .run_control import RunControlHandler
 from .rail_lights import RailLightsHandler
 from .status_bar import StatusBarHandler
+from .task_handler import TaskHandler
 
 
 log = getLogger(__name__)
@@ -85,6 +86,7 @@ class CommandExecutor:
         run_control: RunControlHandler,
         rail_lights: RailLightsHandler,
         status_bar: StatusBarHandler,
+        task_handler: TaskHandler,
         model_utils: Optional[ModelUtils] = None,
         command_note_tracker_provider: Optional[CommandNoteTrackerProvider] = None,
     ) -> None:
@@ -106,6 +108,7 @@ class CommandExecutor:
         self._command_note_tracker_provider = (
             command_note_tracker_provider or _NoteTracker
         )
+        self._task_handler = task_handler
 
     async def execute(self, command_id: str) -> None:
         """Run a given command's execution procedure.
@@ -131,6 +134,7 @@ class CommandExecutor:
             model_utils=self._model_utils,
             status_bar=self._status_bar,
             command_note_adder=note_tracker,
+            task_handler=self._task_handler,
         )
 
         started_at = self._model_utils.get_timestamp()
@@ -214,3 +218,7 @@ class CommandExecutor:
                         type=error_recovery_type,
                     )
                 )
+
+    def cancel_tasks(self, message: str | None = None) -> None:
+        """Cancel all concurrent tasks."""
+        self._task_handler.cancel_all(message=message)

opentrons/protocol_engine/execution/create_queue_worker.py

@@ -17,6 +17,7 @@ from .run_control import RunControlHandler
 from .command_executor import CommandExecutor
 from .queue_worker import QueueWorker
 from .status_bar import StatusBarHandler
+from .task_handler import TaskHandler
 
 
 def create_queue_worker(
@@ -76,7 +77,9 @@ def create_queue_worker(
     rail_lights_handler = RailLightsHandler(
         hardware_api=hardware_api,
     )
-
+    task_handler = TaskHandler(
+        state_store=state_store, action_dispatcher=action_dispatcher
+    )
     status_bar_handler = StatusBarHandler(hardware_api=hardware_api)
 
     command_executor = CommandExecutor(
@@ -93,6 +96,7 @@ def create_queue_worker(
         run_control=run_control_handler,
         rail_lights=rail_lights_handler,
         status_bar=status_bar_handler,
+        task_handler=task_handler,
     )
 
     return QueueWorker(

opentrons/protocol_engine/execution/labware_movement.py

@@ -31,6 +31,7 @@ from ..types import (
     OnLabwareLocation,
     LabwareLocation,
     OnDeckLabwareLocation,
+    GripperMoveType,
 )
 
 if TYPE_CHECKING:
@@ -141,10 +142,16 @@ class LabwareMovementHandler:
             labware_definition = self._state_store.labware.get_definition(labware_id)
 
         from_labware_center = self._state_store.geometry.get_labware_grip_point(
-            labware_definition=labware_definition, location=current_location
+            labware_definition=labware_definition,
+            location=current_location,
+            move_type=GripperMoveType.PICK_UP_LABWARE,
+            user_additional_offset=user_pick_up_offset,
         )
         to_labware_center = self._state_store.geometry.get_labware_grip_point(
-            labware_definition=labware_definition, location=new_location
+            labware_definition=labware_definition,
+            location=new_location,
+            move_type=GripperMoveType.DROP_LABWARE,
+            user_additional_offset=user_drop_offset,
         )
 
         if use_virtual_gripper:
@@ -193,20 +200,10 @@ class LabwareMovementHandler:
         async with self._thermocycler_plate_lifter.lift_plate_for_labware_movement(
             labware_location=current_location
         ):
-            final_offsets = (
-                self._state_store.geometry.get_final_labware_movement_offset_vectors(
-                    from_location=current_location,
-                    to_location=new_location,
-                    additional_pick_up_offset=user_pick_up_offset,
-                    additional_drop_offset=user_drop_offset,
-                    current_labware=labware_definition,
-                )
-            )
             movement_waypoints = get_gripper_labware_movement_waypoints(
                 from_labware_center=from_labware_center,
                 to_labware_center=to_labware_center,
                 gripper_home_z=gripper_homed_position.z,
-                offset_data=final_offsets,
                 post_drop_slide_offset=post_drop_slide_offset,
                 gripper_home_z_offset=gripper_z_offset,
             )