Skip to content

Conversation

@klieret
Copy link
Contributor

@klieret klieret commented Aug 24, 2025

OK this turned into way too many changes. We should call about this.

Biggest change is to have a new tournament class which takes some responsibilities from the game class. The game class should now only run the game and parse outputs, but no longer keep track of rounds etc. (that simply didn't work with the "single player" mode, because we can't just auto-increment rounds when we e.g., just want to evaluate a player against all its previous versions).

However, removing the round logic from the game class meant that we had to refactor how we determine winners (because that was based on the round log files as files). So instead now we get the output of the logs just as return values and determine_winner takes the as a string input.

@john-b-yang john-b-yang merged commit ab34af0 into main Aug 26, 2025
1 check passed
@john-b-yang john-b-yang deleted the single-player-mode branch August 26, 2025 17:51
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.

3 participants