Handling ordered deprecation (#130)

Author: sidnarayanan

packages/hotpotqa/src/aviary/envs/hotpotqa/env.py

@@ -310,13 +310,10 @@ async def step(
         self.state.reward = 0.0
         response_messages = cast(
             Messages,
-            # Ordered since things like search -> lookup need to be run in order.
+            # Non-concurrent since things like search -> lookup need to be run in order.
             # NOTE: Handling tool exceptions here keeps the trajectory going, but I don't
             # think the returned message is useful to the agent/learning. Disabling for now.
-            await self.exec_tool_calls(
-                action,
-                ordered=True,
-            ),
+            await self.exec_tool_calls(action, concurrency=False),
         )
         return response_messages, self.state.reward, self.state.done, False
 

src/aviary/env.py

@@ -166,7 +166,7 @@ async def exec_tool_calls(
         Args:
             message: ToolRequestMessage containing the tool calls.
             concurrency: Flag to set True (default) to concurrently execute tool calls,
-                otherwise set False to execute tools sequentially.
+                otherwise set False to execute tool calls in the provided order.
             handle_tool_exc: Opt-in flag to suppress Exceptions and return them as a
                 ToolResponseMessage.
             handle_invalid_tool_calls: Flag to handle invalid tool calls by returning