Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Petting Zoo environment for LBF #16

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

efosong
Copy link

@efosong efosong commented May 19, 2022

I wrote a PettingZoo implementation of LBF. Perhaps one day we could get it featured on the PettingZoo website, which would be good for visibility.

This might be a bit of an annoying change though since:

  • I left the gym env in, but moved it
  • The interface for the PettingZoo version is quite different (e.g.: ids like Foragaing-8x8-3p-2f-v2 are not used, in preference for just lbforaging_v2 and leave it up to users to specify the environment and report environment parameters in their works)
  • The PettingZoo implementation is pretty much separate from the gym implementation such that changes to one won't automatically appear in the other
  • The predefined agents probably don't work with the PettingZoo env
  • I factored Player out of the code
  • I haven't fully tested grid obs, but I'm about 60% sure it works correctly

Even if you don't want to merge this (really, I made it for my own use), I wanted to start a conversation about a PettingZoo implementation (perhaps wrapping the gym environment more directly?). Or maybe you do want to merge it, but in some different way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant