ElegantRL icon indicating copy to clipboard operation
ElegantRL copied to clipboard

Published 20 hours ago • verified publisher icon AI4Finance-Foundation

Errors on the IsaacGym-Single-Process branch by running demo_IsaacGym.py

Open louise-sawyer opened this issue 2 years ago • 3 comments

  1. In train/run.py, line 35, "if_use_per=args.if_use_per", Argument has no attribute "if_use_per".

  2. In train/evaluator.py, coder should import wandb and write "wandb.init()".

  3. In demo_IsaacGym.py, if I change the Agent such as AgentSAC, there will be an typeerror in agents/AgentBse.py that init() takes 4 positional arguments but 5 were given.

louise-sawyer avatar Jul 22 '22 07:07 louise-sawyer

Thank you for helping us to find these bugs. I am fixing these bugs.

  1. The "IsaacGym-Single-Process branch" don't need attribute if_use_per. We will remove this attribute.
  2. OK. We will add wandb.init() or remove the code use wandb.
  3. init() takes 4 positional arguments but 5 were given. The layer_num cause this bug. We will fix it.

Yonv1943 avatar Jul 23 '22 02:07 Yonv1943

Interested to know whether this bug/issue also happen for this file: https://github.com/AI4Finance-Foundation/ElegantRL/blob/master/demo_IsaacGym.py

YangletLiu avatar Jul 26 '22 05:07 YangletLiu

Interested to know whether this bug/issue also happen for this file: https://github.com/AI4Finance-Foundation/ElegantRL/blob/master/demo_IsaacGym.py

It doesn't also happen. Because when I run the file, it will occur an error in config.py like can't pickle isaacgym._bindings.linux-x86_64.gym_37.Gym objects at line 206 code "env = deepcopy(env)".

louise-sawyer avatar Jul 28 '22 09:07 louise-sawyer