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 old-style loggers #289

Open
jwaa opened this issue May 16, 2021 · 0 comments
Open

Remove old-style loggers #289

jwaa opened this issue May 16, 2021 · 0 comments
Labels
core Work related to the core functionality request A request for a new feature

Comments

@jwaa
Copy link
Member

jwaa commented May 16, 2021

Is your feature request related to a problem? Please describe.
As of v2.1.0 the old logger classes are deprecated, to be replaced by GridWorldLoggerV2 and others. These new loggers included the world state as a State argument, simplifying the implementation of new loggers.

Describe the solution you would like
The old logger classes need to be removed, and the GridWorldLoggerV2 need to become deprecated and renamed to GridWorldLogger. Similar for all other default loggers as well. In a future release then GridWorldLoggerV2 can then be removed again.

Describe alternatives you have considered
N/A

Additional context
The loggers were changed for #202

@jwaa jwaa added core Work related to the core functionality request A request for a new feature labels May 16, 2021
@jwaa jwaa added this to the Release V2.2.0 milestone May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Work related to the core functionality request A request for a new feature
Projects
None yet
Development

No branches or pull requests

1 participant