Skip to content

Added masks for envs - #195

Open
Denys88 wants to merge 2 commits into
masterfrom
DM/masked_envs
Open

Added masks for envs#195
Denys88 wants to merge 2 commits into
masterfrom
DM/masked_envs

Conversation

@Denys88

@Denys88 Denys88 commented Aug 17, 2022

Copy link
Copy Markdown
Owner

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

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

@Denys88

Denys88 commented Aug 17, 2022

Copy link
Copy Markdown
Owner Author

cc @ankurhanda and @ArthurAllshire

@ankurhanda

Copy link
Copy Markdown
Collaborator

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

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.

3 participants