Farama-Foundation / PettingZoo

An API standard for multi-agent reinforcement learning environments, with popular reference environments and related utilities
https://pettingzoo.farama.org
Other
2.45k stars 400 forks source link

Update agilerl version and simplified CNN definition #1182

Closed nicku-a closed 3 months ago

nicku-a commented 4 months ago

Description

Updates agilerl version in tutorials, and also simplifies kernel size definition for multi-agent CNNs.

Fixes # (issue), Depends on # (pull request)

Type of change

Checklist:

nicku-a commented 4 months ago

FYI failing tests correspond to other areas of the codebase (chess test and knights archers zombies docs)

dm-ackerman commented 4 months ago

Hi, I saw the update to AgileRL broke the tutorial a few days ago. Thanks for fixing this! Would you mind updating your branch to the current master? That should fix the broken tests so it can be merged. There will probably be a merge conflict because I pinned the agilerl version to 0.1.19 to stop the errors until the tutorial was fixed. But I think everything else should work fine.

nicku-a commented 3 months ago

Sorry for the delay @dm-ackerman - should be good to go!