Closed feup-jmc closed 2 years ago
Hi, thanks for making this issue! As for the error regarding jacp - I have fixed it now, please pull from master. Everything should run just fine.
Re: should actually be search_space = {"env_name": [args.exp_prefix]} No that is incorrect. The code is actually correct as is. The env name should be set by args.env.
Hello, Mr. Dalal
First of all, I must say I am extremely grateful for your much needed help in such an hour of need.
I have merged with master
and no longer do I get jacp
errors.
So far everything seems fine but I cannot definitively greenlight the changes just yet as my workstation is too busy to have enough memory to run RAPS (RuntimeError: CUDA error: out of memory
) .
There is, however, something I must mention. Without the fixes to search_space
mentioned above, I still get the error mentioned in #1. Does the script run fine on your side without them? If so, I find it very strange that python would have a problem with such a specific part of the program.
I will keep you updated as the situation develops - hopefully the changes to search_space
won't have any negative side effects, but for now I can only hope for the best. 🤞
you can reduce the number of vec_envs to reduce GPU memory usage.
The reason you get the error mentioned in #1 is that you aren't setting exp_prefix to reasonable value (<> is just a placeholder), replace it with a string, say test for example, then it will work fine.
for example:
python experiments/kitchen/dreamer/dreamer_v2_single_task_primitives.py --mode here_no_doodad --exp_prefix test --env microwave
As I said above, don't change the search space code, your change is incorrect.
You are right, replacing <>
in field --exp_prefix
does indeed solve the problem without the need to change the code!
However, the use/behaviour of --exp_prefix
is not made very clear in the README, so I would suggest changing it to make issues such as these less likely to happen in the future (what exactly does the field do? what is the value used for?).
That said, thank you for all the support this far! I am closing the issue as the underlying issue has been resolved.
Hello and once again thanks for the amazing work!
Context
So far, I've managed to install the repo but running tests has proven difficult.
An issue I found early on was that running a sample training command like mentioned in the README didn't work as the program seemed to be unable to remember the environment passed by
--env
. This issue is mentioned in #1, and for the sake of consistency I will use the same command moving forward in these issues:$ python experiments/kitchen/dreamer/dreamer_v2_single_task_primitives.py --mode here_no_doodad --exp_prefix <> --env microwave
With a bit of debugging I think I've managed to solve it, although this issue is not limited to
dreamer_v2_single_task_primitives.py
: https://github.com/mihdalal/raps/blob/23c7d952c9d5e4db6fd96b9a56c1c0373d2bd121/rlkit/experiments/kitchen/dreamer/dreamer_v2_single_task_primitives.py#L117 should actually besearch_space = {"env_name": [args.exp_prefix]}
.Using
sed
for this replacement shows the following to be all the affected files (I can make a PR later if it seems like this issue is not exclusive to me):Issue
With this problem out of the way, however, here is the real focus of this issue - when running the command again after solving the aforementioned problem, this is the output error (with similar results for other
kitchen
training scripts):As I am not too well acquainted with the internals of
MuJoCo
, this would be very difficult for me to debug. I would very thankful if you could provide some insight in how to solve this error.