Fixed an issue with user input agents and games that have a state that does not accept STOP actions.

Author: eriq-augustine

.mypy.ini

@@ -4,3 +4,6 @@ exclude = _test\.py$
 
 strict = false
 disable_error_code = type-arg
+
+[mypy-edq.*]
+ignore_missing_imports = True

pacai/agents/userinput.py

@@ -48,9 +48,13 @@ def get_action_full(self,
             intended_action = state.get_last_agent_action()
             used_user_input = False
 
-        # If the action is illegal, then just stop.
+        # If the action is illegal, then just stop (if legal) or pick a random action.
         if (intended_action not in legal_actions):
-            intended_action = pacai.core.action.STOP
+            if (pacai.core.action.STOP in legal_actions):
+                intended_action = pacai.core.action.STOP
+            else:
+                intended_action = self.rng.choice(legal_actions)
+
             used_user_input = False
 
         return pacai.core.agentaction.AgentAction(intended_action, clear_inputs = used_user_input)

pacai/ui/web.py

@@ -192,7 +192,7 @@ def _handle_request(self, data_handler: typing.Callable) -> None:
         self.end_headers()
 
         if (payload is not None):
-            self.wfile.write(payload)
+            self.wfile.write(payload)  # type: ignore[arg-type]
 
     def _route(self, path: str, params: dict[str, typing.Any]) -> RequestHandlerResult:
         path = path.strip()