From 7701de45ec3499bcb5235c009a62ed360f05eebe Mon Sep 17 00:00:00 2001 From: Max Marrone Date: Fri, 17 May 2024 11:29:28 -0400 Subject: [PATCH] Add a todo comment for redundancy between `get_slice()` and `get_current()`. --- api/src/opentrons/protocol_engine/state/commands.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/api/src/opentrons/protocol_engine/state/commands.py b/api/src/opentrons/protocol_engine/state/commands.py index 8a4098b0a8b4..cd20920c81e7 100644 --- a/api/src/opentrons/protocol_engine/state/commands.py +++ b/api/src/opentrons/protocol_engine/state/commands.py @@ -520,6 +520,8 @@ def get_slice( queued_command_ids = self._state.command_history.get_queue_ids() total_length = self._state.command_history.length() + # TODO(mm, 2024-05-17): This looks like it's attempting to do the same thing + # as self.get_current(), but in a different way. Can we unify them? if cursor is None: if running_command is not None: cursor = running_command.index