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

many bugs #2

Open
Nicolas-Strange opened this issue Jan 9, 2019 · 6 comments
Open

many bugs #2

Nicolas-Strange opened this issue Jan 9, 2019 · 6 comments

Comments

@Nicolas-Strange
Copy link

Hello,
thanks you for sharing your code. I am very interested in your model but there is no way to make it working.
I have fixed a lot of bugs (I cannot summarize all since there are a lot) but there are many other that would require to re-write some parts.
Do you have an up-to-date version that works?
Thanks again.

@davidleejy
Copy link

Works for me. Cloned and running run_molecule.py today. Perhaps you'd like to share the first error you encounter?

@Frank-LIU-520
Copy link

Works for me. Cloned and running run_molecule.py today. Perhaps you'd like to share the first error you encounter?

Did u try it with python 3.5 or python 3.6? Any detail of the version compability of all packages?
I even cannot install it with warning "You appear to be missing MuJoCo. We expected to find the file here: C:\Users\mario.mujoco\mujoco200".
Any suggestions?Thanks so much.

@mamengyiyi
Copy link

mamengyiyi commented Apr 3, 2019

Works for me. Cloned and running run_molecule.py today. Perhaps you'd like to share the first error you encounter?

Did u try it with python 3.5 or python 3.6? Any detail of the version compability of all packages?
I even cannot install it with warning "You appear to be missing MuJoCo. We expected to find the file here: C:\Users\mario.mujoco\mujoco200".
Any suggestions?Thanks so much.

MoJoCo is not used in this env. Just follow the instructions in readme.md, and move the /rl-baselines/baselines to the same directory with run_molecule.py . That's worked for me

@sschangi
Copy link

I am following the readme.md instruction, and moved the rl-baselines/baselines to the same directory as run_molecule.py. Unfortunately, I am still getting the same MuJoCo error. Would anyone help me with that?

@davebiagioni
Copy link

I think many of the issues arise from version compatibility. It would be great if the authors could put a requirements.txt or environment.yaml file at the top of the repo so that installation could be more reproducible.

@sschangi @mamengyiyi As far as mujoco goes, there is a simple workaround: remove gym from the install_requirements in the setup.py file:

cd rl-baselines
# Remove gym from install_requirements list in setup.py
pip install -e .
pip install gym

@chetankm1992
Copy link

@davidleejy at least this error (

ERROR: Failed building wheel for mujoco-py

) was removed after following your comments. Thanks.

I think many of the issues arise from version compatibility. It would be great if the authors could put a requirements.txt or environment.yaml file at the top of the repo so that installation could be more reproducible.

@sschangi @mamengyiyi As far as mujoco goes, there is a simple workaround: remove gym from the install_requirements in the setup.py file:

cd rl-baselines
# Remove gym from install_requirements list in setup.py
pip install -e .
pip install gym

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

No branches or pull requests

7 participants