The ladder no longer removes players from the player pool. It instead sets an in_game flag. In addition, instead of checking against player_pool, it checks against the available_players.
The purpose of this change is so that way when Elo rankings are calculated for generating stats, they are not affected by the fact that some players may be in games.
Test Cases
agent_tests/base_agent_test.py
Tests that in_game is defaulted to False
ladder_tests/base_ladder_test.py
When two players with in_game = False, the available_players() function returns both of them
When one players with in_game = False, the available_players() function returns one of them
Addresses Issue #162
Updates
The ladder no longer removes players from the player pool. It instead sets an
in_gameflag. In addition, instead of checking againstplayer_pool, it checks against theavailable_players. The purpose of this change is so that way when Elo rankings are calculated for generating stats, they are not affected by the fact that some players may be in games.Test Cases
agent_tests/base_agent_test.pyin_gameis defaulted toFalseladder_tests/base_ladder_test.pyin_game = False, theavailable_players()function returns both of themin_game = False, theavailable_players()function returns one of them