Skip to content

Commit

Permalink
typo in comment
Browse files Browse the repository at this point in the history
  • Loading branch information
andySigler authored and ryanthecoder committed Jul 22, 2024
1 parent e4c56a6 commit db48d60
Showing 1 changed file with 31 additions and 0 deletions.
31 changes: 31 additions & 0 deletions api/src/opentrons/hardware_control/ot3api.py
Original file line number Diff line number Diff line change
Expand Up @@ -1935,6 +1935,37 @@ async def _move_to_plunger_bottom(
acquire_lock=acquire_lock,
)

async def _move_to_plunger_top_for_liquid_probe(
self,
mount: OT3Mount,
rate: float,
acquire_lock: bool = True,
) -> None:
"""
Move an instrument's plunger to the top, to prepare for a following
liquid probe action.
The plunger backlash distance (mm) is used to ensure the plunger is pre-loaded
in the downward direction. This means that the final position will not be
the plunger's configured "top" position, but "top" plus the "backlashDistance".
"""
max_speeds = self.config.motion_settings.default_max_speed
speed = max_speeds[self.gantry_load][OT3AxisKind.P]
instrument = self._pipette_handler.get_pipette(mount)
top_plunger_pos = target_position_from_plunger(
OT3Mount.from_mount(mount),
instrument.plunger_positions.top,
self._current_position,
)
target_pos = top_plunger_pos.copy()
target_pos[Axis.of_main_tool_actuator(mount)] += instrument.backlash_distance
await self._move(top_plunger_pos, speed=speed * rate, acquire_lock=acquire_lock)
# NOTE: This should ALWAYS be moving DOWN.
# There should never be a time that this function is called and
# the plunger doesn't physically move DOWN.
# This is to make sure we are always engaged at the beginning of liquid-probe.
await self._move(target_pos, speed=speed * rate, acquire_lock=acquire_lock)

async def configure_for_volume(
self, mount: Union[top_types.Mount, OT3Mount], volume: float
) -> None:
Expand Down

0 comments on commit db48d60

Please sign in to comment.