eugenevinitsky / sequential_social_dilemma_games

Repo for reproduction of sequential social dilemmas
MIT License
384 stars 134 forks source link

refactor, bug: refactored the environments to separate the map of the world from the agent positions. #124

Closed eugenevinitsky closed 5 years ago

eugenevinitsky commented 5 years ago

In the current code base the map of the world and the agent positions are tracked on the same map. As a consequence, whenever an agent covers a position we have to keep track of what is being covered. This leads to a lot of unnecessary code. Agent positions are now tracked by the agents and the map of the world is kept in a variable called world_map.

Resolves #83 This is written Resolves #82 This is written Resolves #111 These are written Resolves #110 It is based on underlying map state which is now done correctly now Resolves #112 These tests are written Resolves #117 hidden_cells is removed Resolves #121 this PR just does this Resolves #119 by getting rid of the reservation objects Resolves #108 by getting rid of hidden cells