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.

Sign up for GitHub

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

Jump to bottom

Added masks for envs #195

Open

Denys88 wants to merge 2 commits into master from DM/masked_envs

Owner

Denys88 commented Aug 17, 2022 •

edited

Loading

It was requested in this ticket: #194
Two things should be done:

env_info returned from the vec_env should have 'env_masks' : True
method get_env_masks(self) should be implemented

DenSumy added 2 commits

August 16, 2022 16:26


          added masked envs

394ff20


          env masks

2f91771

Owner Author

Denys88 commented Aug 17, 2022

cc @ankurhanda and @ArthurAllshire

Collaborator

ankurhanda commented Aug 17, 2022

Would be good to have a unit test on this as this feature is important.

Denys88 mentioned this pull request

Evaluation worker feature #192

Open

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

Labels

None yet