From 93d9860ad6d3fbf0e1cad39e3625ad87249fbeb9 Mon Sep 17 00:00:00 2001 From: Max Marrone Date: Thu, 17 Oct 2024 13:29:11 -0400 Subject: [PATCH] Move simulation-specific thing to tip_drop_moves(). I don't understand what this does, and given that it's a state update, it seems like it should not live in tip_drop_moves(). But tip_pickup_moves() has the same thing, so I presume we should treat it the same way. --- api/src/opentrons/hardware_control/ot3api.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/api/src/opentrons/hardware_control/ot3api.py b/api/src/opentrons/hardware_control/ot3api.py index 52be73119a3a..54c776d39a3e 100644 --- a/api/src/opentrons/hardware_control/ot3api.py +++ b/api/src/opentrons/hardware_control/ot3api.py @@ -2319,6 +2319,10 @@ async def tip_drop_moves( if home_after: await self._home([Axis.by_mount(mount)]) + # call this in case we're simulating: + if isinstance(self._backend, OT3Simulator): + self._backend._update_tip_state(realmount, False) + async def drop_tip( self, mount: Union[top_types.Mount, OT3Mount], home_after: bool = False ) -> None: @@ -2336,10 +2340,6 @@ async def drop_tip( self.set_current_tiprack_diameter(mount, 0.0) await self.remove_tip(mount) - # call this in case we're simulating: - if isinstance(self._backend, OT3Simulator): - self._backend._update_tip_state(realmount, False) - async def clean_up(self) -> None: """Get the API ready to stop cleanly.""" await self._backend.clean_up()