-
-
Notifications
You must be signed in to change notification settings - Fork 611
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
Added a stochastic action wrapper along with its test #355
Added a stochastic action wrapper along with its test #355
Conversation
@BolunDai0216 Why should we add this as a random action in minigrid rather than gymnasium? |
It is to make the library more complete. Additionally, in the future, we can update the wrapper to specifically handle random action needs in Miniworld instead. If it is in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can include this in minigrid and move it to gymnasium later if we think that it is important
Description
This PR adds a new wrapper that makes the action stochastic. In the wrapper, the user can specify the probability that a random action is applied, along with what kind of random action is applied (it can either be a truly random action sampled from the action space or a user-specified action).
Type of change
Checklist:
pre-commit
checks withpre-commit run --all-files
(seeCONTRIBUTING.md
instructions to set it up)