From 61ed31a45ae802091f1128f7d1ecec12e378fd01 Mon Sep 17 00:00:00 2001 From: Max Marrone Date: Wed, 13 Sep 2023 10:32:57 -0400 Subject: [PATCH] Remove bad todo comment. --- api/src/opentrons/protocol_engine/protocol_engine.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/api/src/opentrons/protocol_engine/protocol_engine.py b/api/src/opentrons/protocol_engine/protocol_engine.py index 9dd81be78dc..fa851ec29e7 100644 --- a/api/src/opentrons/protocol_engine/protocol_engine.py +++ b/api/src/opentrons/protocol_engine/protocol_engine.py @@ -347,8 +347,8 @@ async def finish( # By the time E-stop error has gotten to us, it may have been wrapped in other errors, # so we need to unwrap them to uncover the E-stop error's inner beauty. # - # TODO(mm, 2023-09-12): Do we need to scan the exception tree like this? Instead, can we - # directly inspect the E-stop state through self._hardware_api.get_estop_state()? + # We don't use self._hardware_api.get_estop_state() because the E-stop may have been + # released by the time we get here. if isinstance(error, EnumeratedError) and self._code_in_error_tree( root_error=error, code=ErrorCodes.E_STOP_ACTIVATED ):