Update multi_agent.py

multi_agent.py

@@ -2,51 +2,51 @@ import chess, chess.svg, math
 from autogen import ConversableAgent, register_function
 from typing_extensions import Annotated
 
+def get_legal_moves() -> Annotated[str, "A list of legal moves in UCI format"]:
+    return "Possible moves are: " + ",".join(
+        [str(move) for move in board.legal_moves]
+    )
+
+def make_move(move: Annotated[str, "A move in UCI format."]) -> Annotated[str, "Result of the move."]:
+    move = chess.Move.from_uci(move)
+    board.push_uci(str(move))
+    global made_move
+    made_move = True
+
+    board_svgs.append(chess.svg.board(
+        board,
+        arrows=[(move.from_square, move.to_square)],
+        fill={move.from_square: "gray"},
+        size=250
+    ))
+    
+    piece = board.piece_at(move.to_square)
+    piece_symbol = piece.unicode_symbol()
+    piece_name = (
+        chess.piece_name(piece.piece_type).capitalize()
+        if piece_symbol.isupper()
+        else chess.piece_name(piece.piece_type)
+    )
+    
+    return f"Moved {piece_name} ({piece_symbol}) from "\
+           f"{chess.SQUARE_NAMES[move.from_square]} to "\
+           f"{chess.SQUARE_NAMES[move.to_square]}."
+
+def check_made_move(msg):
+    global made_move
+    
+    if made_move:
+        made_move = False
+        return True
+    else:
+        return False
+
 class MultiAgent:
     made_move = False
     
     board = chess.Board()
     board_svgs = []
-    
-    def get_legal_moves(self) -> Annotated[str, "A list of legal moves in UCI format"]:
-        return "Possible moves are: " + ",".join(
-            [str(move) for move in board.legal_moves]
-        )
-    
-    def make_move(self, move: Annotated[str, "A move in UCI format."]) -> Annotated[str, "Result of the move."]:
-        move = chess.Move.from_uci(move)
-        board.push_uci(str(move))
-        global made_move
-        made_move = True
-    
-        board_svgs.append(chess.svg.board(
-            board,
-            arrows=[(move.from_square, move.to_square)],
-            fill={move.from_square: "gray"},
-            size=250
-        ))
-        
-        piece = board.piece_at(move.to_square)
-        piece_symbol = piece.unicode_symbol()
-        piece_name = (
-            chess.piece_name(piece.piece_type).capitalize()
-            if piece_symbol.isupper()
-            else chess.piece_name(piece.piece_type)
-        )
-        
-        return f"Moved {piece_name} ({piece_symbol}) from "\
-               f"{chess.SQUARE_NAMES[move.from_square]} to "\
-               f"{chess.SQUARE_NAMES[move.to_square]}."
-    
-    def check_made_move(self, msg):
-        global made_move
-        
-        if made_move:
-            made_move = False
-            return True
-        else:
-            return False
-    
+
     def get_num_turns(self, num_moves):
         # Each turn includes two moves (one by each player)
         # The first move by player black kicks off the chat
@@ -58,7 +58,7 @@ class MultiAgent:
             num_turns += 1
             
         return num_turns
-        
+    
     def run_multi_agent(self, llm_white, llm_black, num_moves):   
         llm_config_white = {"model": llm_white}
         llm_config_black = {"model": llm_black}
@@ -66,7 +66,7 @@ class MultiAgent:
         board_proxy = ConversableAgent(
             name="Board Proxy",
             llm_config=False,
-            is_termination_msg=self.check_made_move,
+            is_termination_msg=check_made_move,
             default_auto_reply="Please make a move.",
             human_input_mode="NEVER",
         )
@@ -91,7 +91,7 @@ class MultiAgent:
         
         for caller in [player_white, player_black]:
             register_function(
-                self.get_legal_moves,
+                get_legal_moves,
                 caller=caller,
                 executor=board_proxy,
                 name="get_legal_moves",
@@ -99,7 +99,7 @@ class MultiAgent:
             )
         
             register_function(
-                self.make_move,
+                make_move,
                 caller=caller,
                 executor=board_proxy,
                 name="make_move",