From fdb1b0be8f9e2a38e7c21c63ad79045443de44d9 Mon Sep 17 00:00:00 2001 From: Sanniti Date: Thu, 12 Dec 2024 16:40:59 -0500 Subject: [PATCH] docstrings and comments cleanup --- .../core/engine/transfer_components_executor.py | 2 +- .../core/engine/test_transfer_components_executor.py | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/api/src/opentrons/protocol_api/core/engine/transfer_components_executor.py b/api/src/opentrons/protocol_api/core/engine/transfer_components_executor.py index 3663501961c..2e4dae08fab 100644 --- a/api/src/opentrons/protocol_api/core/engine/transfer_components_executor.py +++ b/api/src/opentrons/protocol_api/core/engine/transfer_components_executor.py @@ -237,7 +237,7 @@ def _add_air_gap(self, air_gap_volume: float) -> None: def _remove_air_gap(self, location: Location, volume: float) -> None: """Remove a previously added air gap.""" dispense_props = self._transfer_properties.dispense - # To err on the side of caution, the maximum flow rate should be air_gap_volume per second + # The maximum flow rate should be air_gap_volume per second flow_rate = min( dispense_props.flow_rate_by_volume.get_for_volume(volume), volume ) diff --git a/api/tests/opentrons/protocol_api/core/engine/test_transfer_components_executor.py b/api/tests/opentrons/protocol_api/core/engine/test_transfer_components_executor.py index 3c57adb30da..7be520b173b 100644 --- a/api/tests/opentrons/protocol_api/core/engine/test_transfer_components_executor.py +++ b/api/tests/opentrons/protocol_api/core/engine/test_transfer_components_executor.py @@ -62,7 +62,7 @@ def test_submerge( mock_instrument_core: InstrumentCore, sample_transfer_props: TransferProperties, ) -> None: - """Should perform the expected aspiration steps.""" + """Should perform the expected submerge steps.""" source_well = decoy.mock(cls=WellCore) well_top_point = Point(1, 2, 3) well_bottom_point = Point(4, 5, 6) @@ -151,7 +151,7 @@ def test_aspirate_and_wait_skips_delay( mock_instrument_core: InstrumentCore, sample_transfer_props: TransferProperties, ) -> None: - """It should execute an aspirate and a delay according to properties.""" + """It should skip the wait after aspirate.""" sample_transfer_props.aspirate.delay.enabled = False source_well = decoy.mock(cls=WellCore) @@ -206,7 +206,7 @@ def test_dispense_and_wait_skips_delay( mock_instrument_core: InstrumentCore, sample_transfer_props: TransferProperties, ) -> None: - """It should execute an aspirate and a delay according to properties.""" + """It should skip the wait after dispense.""" sample_transfer_props.dispense.delay.enabled = False source_well = decoy.mock(cls=WellCore)