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

[Feature Request] update CMDP/Wrapper API to gymnasium API #265

Closed
1 of 2 tasks
luigiberducci opened this issue Aug 9, 2023 · 2 comments
Closed
1 of 2 tasks

[Feature Request] update CMDP/Wrapper API to gymnasium API #265

luigiberducci opened this issue Aug 9, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@luigiberducci
Copy link

Required prerequisites

Motivation

I had some issue with the fact that the reset method does not match the signature of gymnasium API.
In particular, the argument options is missing.

Solution

Update the signature as
reset( self, *, seed: int | None = None, options: dict[str, Any] | None = None, ) -> tuple[ObsType, dict[str, Any]]:

Alternatives

No response

Additional context

No response

@luigiberducci luigiberducci added the enhancement New feature or request label Aug 9, 2023
@Gaiejj
Copy link
Member

Gaiejj commented Aug 10, 2023

Pull request #266 would support this new interface. You can check whether it meets your requirement and feel free to review it.

@luigiberducci
Copy link
Author

Thanks, I will look into it!

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

No branches or pull requests

2 participants