-
Notifications
You must be signed in to change notification settings - Fork 42
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_manager' #1
Comments
check this: rll/rllab#67 |
@tigerneil : Thanks! |
I have it fixed on new version (merged from openai/rllab), but I am cleaning up the new codes (added new features) and have not committed. For now, you may install older gym versions. If I recall correctly, gym==5.6 for example works. Issues with monitor are also described here: https://pypi.python.org/pypi/gym. |
Replacing rllabplusplus/rllab/envs/gym_env.py by the original file from openai/rllab works for sure however I did not try downgrading to older gym yet. |
@shaneshixiang and @alexpashevich : |
Hi,
I successfully installed rllabplusplus, and the trpo_cartpole.py example worked fine, but I'm getting an error with algo_gym_stub.py
python algo_gym_stub.py --exp=HalfCheetah-v1
Traceback (most recent call last):
File "algo_gym_stub.py", line 1, in
from sandbox.rocky.tf.launchers.launcher_utils import FLAGS, get_env_info, get_annotations_string
File "/home/RL-work/code/rllab/rllabplusplus/sandbox/rocky/tf/launchers/launcher_utils.py", line 10, in
from rllab.envs.gym_env import GymEnv
File "/home/RL-work/code/rllab/rllabplusplus/rllab/envs/gym_env.py", line 5, in
from gym.monitoring import monitor_manager
ImportError: cannot import name 'monitor_manager'
Do you know how to fix this problem?
The text was updated successfully, but these errors were encountered: