Skip to content

Single player mode initial version#33

Merged
john-b-yang merged 8 commits into
mainfrom
single-player-mode
Aug 26, 2025
Merged

Single player mode initial version#33
john-b-yang merged 8 commits into
mainfrom
single-player-mode

Conversation

@klieret

@klieret klieret commented Aug 24, 2025

Copy link
Copy Markdown
Contributor

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.

2 participants