Originally Mark was intended to effectively identify a player. Now we have PlayerID for that, and Mark is not needed anymore, given that we don't allow players to choose marks (we can just use x for playero, and o for player1).
Implementing this issue includes renaming player_x/player_y/act_queue_px/etc. to player_0/player_1/act_queue_p0, for example, in test/test_game.py. It also makes expressions like state.players[1].id meaningless, requiring them to be replaced with 1.
Originally
Mark
was intended to effectively identify a player. Now we havePlayerID
for that, andMark
is not needed anymore, given that we don't allow players to choose marks (we can just usex
for playero, ando
for player1).Implementing this issue includes renaming
player_x
/player_y
/act_queue_px
/etc. toplayer_0
/player_1
/act_queue_p0
, for example, intest/test_game.py
. It also makes expressions likestate.players[1].id
meaningless, requiring them to be replaced with1
.Note: done in Rust.