Skip to content

Conversation

@dysonreturns
Copy link

  • The example observer file needs it's path to be cast to string, otherwise we get a TypeError.
  • ObserverAI dependend on deprecated self._client, self._game_data, self._game_info fields. They shoudl now be accessed directly.
  • Additionally, observing as Everyone (player 0) broke, because it had no race. When player is 0, we assign the race of Random and it has no concerns.

dysonreturns added 5 commits January 17, 2025 12:32
…are based on.

Latest 5.0.14 (94137) is bugged on the API, i.e. missing a lot of Upgrades.
This commit allows sc2-techtree to this version, by supplying `run_game` with: `, sc2_version="5.0.14.93333")`
@BurnySc2
Copy link
Owner

Thanks for putting effort into it.

I would like to add at least one test case, too. Because otherwise the ObserverAI class is going to lack behind the BotAI class forever and only is being manually tested.
This is especially bad since it inherits from BotAIInternal

Can you think of a (simple) strategy to test it? Will a real bot replay be needed? Perhaps one can be created during the test, and then run through the ObserverAI to validate. What if the replay is from an older patch?
I am just asking for suggestions, I try to write one myself. I am not expecting you to add one, but that would be awesome of course.

@BurnySc2 BurnySc2 merged commit 7a1c6f3 into BurnySc2:develop Jun 27, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants