Repairs ObserverAI by removing deprecated field usage and assigning random Race for player_id 0(zero)

Author: dysonreturns

sc2/bot_ai_internal.py

@@ -476,10 +476,16 @@ def _prepare_start(
         self.realtime: bool = realtime
         self.base_build: int = base_build
 
-        self.race: Race = Race(self.game_info.player_races[self.player_id])
+        if self.player_id > 0:
+            self.race: Race = Race(self.game_info.player_races[self.player_id])
 
-        if len(self.game_info.player_races) == 2:
-            self.enemy_race: Race = Race(self.game_info.player_races[3 - self.player_id])
+            if len(self.game_info.player_races) == 2:
+                self.enemy_race: Race = Race(self.game_info.player_races[3 - self.player_id])
+        else:
+            # Observer
+            self.race: Race = Race.Random
+
+        self._distances_override_functions(self.distance_calculation_method)
 
         self._distances_override_functions(self.distance_calculation_method)
 

sc2/observer_ai.py

@@ -6,21 +6,14 @@
 
 from __future__ import annotations
 
-from typing import TYPE_CHECKING
-
 from sc2.bot_ai_internal import BotAIInternal
 from sc2.data import Alert, Result
-from sc2.game_data import GameData
 from sc2.ids.ability_id import AbilityId
 from sc2.ids.upgrade_id import UpgradeId
 from sc2.position import Point2
 from sc2.unit import Unit
 from sc2.units import Units
 
-if TYPE_CHECKING:
-    from sc2.client import Client
-    from sc2.game_info import GameInfo
-
 
 class ObserverAI(BotAIInternal):
     """Base class for bots."""
@@ -36,21 +29,6 @@ def time_formatted(self) -> str:
         t = self.time
         return f"{int(t // 60):02}:{int(t % 60):02}"
 
-    @property
-    def game_info(self) -> GameInfo:
-        """See game_info.py"""
-        return self._game_info
-
-    @property
-    def game_data(self) -> GameData:
-        """See game_data.py"""
-        return self._game_data
-
-    @property
-    def client(self) -> Client:
-        """See client.py"""
-        return self._client
-
     def alert(self, alert_code: Alert) -> bool:
         """
         Check if alert is triggered in the current step.