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

[Proposal] Add wind attribute for stochastic environments #228

Open
pseudo-rnd-thoughts opened this issue Sep 8, 2022 · 4 comments
Open

Comments

@pseudo-rnd-thoughts
Copy link
Member

In Distributional Reinforcement Learning with Quantile Regression, they propose a testing environment where wind is added to the environment to make a gridworld problem stochastic.

I propose a new environment or minigrid environment argument to add this wind to any environment
I favour the new environment approach as I think this is a feature that other environment would not want or need

@siddarth-c
Copy link

I have started working on this

@siddarth-c
Copy link

siddarth-c commented Sep 6, 2023

Sorry for the delay! I got busy with something else. Will a wrapper work for this?
Something like: It takes in a list of size [1 x grid_width] as input, where each element represents the probability of the agent being pushed up north by one square?

@pseudo-rnd-thoughts
Copy link
Member Author

Yes, a wrapper would work but this would require modifying the actual environment data which a wrapper doesn't normally operate on.
If you have an implementation then it shouldn't be too hard to move the implementation from a wrapper to an environment or vice-versa

@riiswa
Copy link

riiswa commented Sep 20, 2023

@siddarth-c are you still on it ?

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

No branches or pull requests

3 participants