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

Remove AbstractMarkovEnv and AbstractZeroSumEnv #43

Open
zsunberg opened this issue Jan 2, 2021 · 3 comments
Open

Remove AbstractMarkovEnv and AbstractZeroSumEnv #43

zsunberg opened this issue Jan 2, 2021 · 3 comments
Milestone

Comments

@zsunberg
Copy link
Member

zsunberg commented Jan 2, 2021

After discussion in #40 , I think there is good reason to get rid of AbstractMarkovEnv and AbstractZeroSumEnv. I think we will be able to eventually handle multiplayer environments with a single AbstractEnv. Expect more discussion in #36 .

@zsunberg zsunberg added this to the v0.3 milestone Jan 2, 2021
@zsunberg
Copy link
Member Author

zsunberg commented Jan 2, 2021

As a consequence of this, player will become an optional interface function

@jonathan-laurent
Copy link

What is the standard idiom now to know whether an environment is a one-player environment, a zero-sum environment or a multi-player environment?

@zsunberg
Copy link
Member Author

zsunberg commented Jan 16, 2021

Hey @jonathan-laurent glad to hear from you :). Currently there is none because I don't have the experience necessary to design one. I will submit a PR with my suggestion shortly, but I am hoping that we can work together to come up with something good!

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

No branches or pull requests

2 participants