-
Notifications
You must be signed in to change notification settings - Fork 800
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
Cannot import name 'monitor' for trpo_gym #67
Comments
Sorry for the late reply! There has been some updates made to the gym repo. Should be fixed in 142b68d. |
Oh dear :( , seems like it might be broken again?
|
Fixed! |
Fan Q :) By the way, have you seen this LEARNING TO REINFORCEMENT LEARN Be really cool if you released your RL^2 code? |
Hi @ajaytalati, yes we are working on the code release :) It has been delayed mostly because the first-order modifications to TRPO is waiting to be released separately. The learning to reinforcement learn paper has very similar ideas (except the learning algorithm used), so if you'd like to work on any follow-up ideas, your linked repo could be a good starting point. |
Hi, |
@serteckian Make sure you are using the latest rllab code? |
This fixed things for me: |
I successfully installed rllab, and the trpo_cartpole.py example worked fine, but I'm getting an error with trpo_gym.py:
I was looking at the latest Gym source code and
gym.monitoring
is definitely there but I can't find anymonitor
method or class.The text was updated successfully, but these errors were encountered: