Skip to content

Commit

Permalink
Move simulation-specific thing to tip_drop_moves().
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
SyntaxColoring committed Oct 17, 2024
1 parent c45fd89 commit 9591f89
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions api/src/opentrons/hardware_control/ot3api.py
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand All @@ -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()
Expand Down

0 comments on commit 9591f89

Please sign in to comment.