google-deepmind / meltingpot

A suite of test scenarios for multi-agent reinforcement learning.
Apache License 2.0
614 stars 122 forks source link

Adjust change since Melting Pot 2.0 #109

Closed ninell-oldenburg closed 1 year ago

ninell-oldenburg commented 1 year ago

This at least made it work for me, I was thinking it was somehow connected to the introduction of roles in 2.0

google-cla[bot] commented 1 year ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

duenez commented 1 year ago

This doesn't seem to have been updated to use the default player roles in config. Please update and I'll approve this change. :) Thanks a lot!

ninell-oldenburg commented 1 year ago

Sure, thanks for the reminder :)

duenez commented 1 year ago

I believe this is already at head? I'm closing the PR, but feel free to re-open if I got it wrong.