-
Notifications
You must be signed in to change notification settings - Fork 460
better support for custom environments #20
Comments
Thanks Zach, I think that this is a good suggestion. |
agree, would love this. I wanted to try using this with https://github.com/jtoy/sensenet as its openai compatible, but there were too many files to modify. |
I added a new method for specifying which environment to use in the case that the environment follows the interface provided by openai gym. In a preset, you can now specify the environment via: I will leave this issue open because it would still be nice to configure coach without modifying |
@zach-nervana Great, I'll test it out and let you know. |
@zach-nervana could you update the docs (https://nervanasystems.github.io/coach/contributing/add_env/index.html) to reflect your change? Thanks, |
thanks @saddy001, it is now updated in https://nervanasystems.github.io/coach/contributing/add_env/index.html as you suggested. |
At the moment, the following steps are required to use an environment not included with coach:
environments/__init__.py
environments/__init__.py
configurations.py
presets.py
these steps are from here: http://coach.nervanasys.com/contributing/add_env/index.html
Many of these steps require modifying coach internal files. In order to track these changes in a version control system, coach itself must be forked and modified. This is inconvenient for multiple reasons:
pip install coach --upgrade
to get upgrades.Additionally, this pattern of modifying coaches internals in order to add environments or run experiments is not very new user-friendly.
Here is an example from
rllab
documentation:The user is required to create a new environment wrapper following their environment pattern similar to our step 1. After that however, the user only needs to write a simple python script with a few imports and some configuration.
Looking through
coach.py
, it looks like it shouldn't be terribly difficult to provide a similar interface, though I am not entirely sure. What do you guys think?The text was updated successfully, but these errors were encountered: