rl_graph_generation icon indicating copy to clipboard operation
rl_graph_generation copied to clipboard

many bugs

Open Nicolas-Strange opened this issue 6 years ago • 6 comments

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.

Nicolas-Strange avatar Jan 09 '19 06:01 Nicolas-Strange

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

davidleejy avatar Mar 25 '19 08:03 davidleejy

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.

Frank-LIU-520 avatar Apr 02 '19 11:04 Frank-LIU-520

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

mamengyiyi avatar Apr 03 '19 04:04 mamengyiyi

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?

sschangi avatar Jan 17 '20 18:01 sschangi

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

davebiagioni avatar Jun 01 '20 18:06 davebiagioni

@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

chetankm1992 avatar Jun 11 '20 19:06 chetankm1992