Fixed human player

app.py

@@ -232,25 +232,32 @@ def setup_player_config(
     player_type: str, player_model: str, player_id: str
 ) -> Dict[str, Any]:
     """Map dropdown selection to agent config for the runner."""
-    if player_type == "random_bot":
+    # Create a temporary config to get the display-to-key mapping
+    temp_config = create_player_config()
+    display_to_key = {v: k for k, v in temp_config["player_config"]["player_type_display"].items()}
+
+    # Map display label back to internal key
+    internal_key = display_to_key.get(player_type, player_type)
+
+    if internal_key == "random_bot":
         return {"type": "random"}
 
-    if player_type == "human":
+    if internal_key == "human":
         return {"type": "human"}
 
     if (
-        player_type
+        internal_key
         and (
-            player_type.startswith("llm_")
-            or player_type.startswith("hf_")
+            internal_key.startswith("llm_")
+            or internal_key.startswith("hf_")
         )
     ):
-        model_id = player_type.split("_", 1)[1]
+        model_id = internal_key.split("_", 1)[1]
         if BACKEND_SYSTEM_AVAILABLE and model_id in HUGGINGFACE_MODELS:
             return {"type": "llm", "model": model_id}
 
     if (
-        player_type == "llm"
+        internal_key == "llm"
         and player_model in HUGGINGFACE_MODELS
         and BACKEND_SYSTEM_AVAILABLE
     ):
@@ -544,14 +551,18 @@ with gr.Blocks() as interface:
 
         def player_selector_block(label: str):
             gr.Markdown(f"### {label}")
-            choices_pairs = [
-                (key, config["player_config"]["player_type_display"][key])
+            # Create display choices (what user sees)
+            display_choices = [
+                config["player_config"]["player_type_display"][key]
                 for key in config["player_config"]["player_types"]
             ]
+            # Set default to first display choice
+            default_choice = display_choices[0] if display_choices else None
+
             dd_type = gr.Dropdown(
-                choices=choices_pairs,
+                choices=display_choices,
                 label=f"{label} Type",
-                value=choices_pairs[0][0] if choices_pairs else None,
+                value=default_choice,
             )
             dd_model = gr.Dropdown(
                 choices=config["player_config"]["available_models"],
@@ -565,13 +576,17 @@ with gr.Blocks() as interface:
             p2_type, p2_model = player_selector_block("Player 1")
 
         def _vis(player_type: str):
+            # Map display label back to internal key
+            display_to_key = {v: k for k, v in config["player_config"]["player_type_display"].items()}
+            internal_key = display_to_key.get(player_type, player_type)
+
             is_llm = (
-                player_type == "llm"
+                internal_key == "llm"
                 or (
-                    player_type
+                    internal_key
                     and (
-                        player_type.startswith("llm_")
-                        or player_type.startswith("hf_")
+                        internal_key.startswith("llm_")
+                        or internal_key.startswith("hf_")
                     )
                 )
             )
@@ -639,7 +654,12 @@ with gr.Blocks() as interface:
 
         def check_for_human_players(p1_type, p2_type):
             """Show/hide interactive controls based on player types."""
-            has_human = (p1_type == "human" or p2_type == "human")
+            # Map display labels back to internal keys
+            display_to_key = {v: k for k, v in config["player_config"]["player_type_display"].items()}
+            p1_key = display_to_key.get(p1_type, p1_type)
+            p2_key = display_to_key.get(p2_type, p2_type)
+
+            has_human = (p1_key == "human" or p2_key == "human")
             return (
                 gr.update(visible=has_human),  # interactive_panel
                 gr.update(visible=has_human),  # start_interactive_btn
@@ -676,20 +696,27 @@ with gr.Blocks() as interface:
                 from ui.gradio_config_generator import start_game_interactive
                 import time
 
+                # Map display labels back to internal keys
+                display_to_key = {v: k for k, v in config["player_config"]["player_type_display"].items()}
+                p1_key = display_to_key.get(p1_type, p1_type)
+                p2_key = display_to_key.get(p2_type, p2_type)
+
+                # Map display game name back to internal key if needed
+                game_display_to_key = config.get("game_display_to_key", {})
+                internal_game = game_display_to_key.get(game_name, game_name)
+
                 # Use timestamp as seed
                 seed = int(time.time() * 1000) % (2**31 - 1)
 
                 log, state, legal_p0, legal_p1 = start_game_interactive(
-                    game_name=game_name,
-                    player1_type=p1_type,
-                    player2_type=p2_type,
+                    game_name=internal_game,
+                    player1_type=p1_key,
+                    player2_type=p2_key,
                     player1_model=p1_model,
                     player2_model=p2_model,
                     rounds=rounds,
                     seed=seed,
-                )
-
-                # Update dropdowns with legal actions
+                )                # Update dropdowns with legal actions
                 p0_choices = [(action, label) for action, label in legal_p0]
                 p1_choices = [(action, label) for action, label in legal_p1]