Skip to content

[Bug Report] rl_games workflow's log_root_path is not the absolute path when pbt is not used #3530

@ozhanozen

Description

@ozhanozen

After the addition of pbt logging support, rl_games workflow's log_root_path is not the absolute path anymore, if pbt is not used. This breaks functionality in Ray, and potentially in other places.

Steps to reproduce

If you have a look at the output [INFO] Logging experiment in directory: ..., you will see that it is not the absolute path.

System Info

Describe the characteristic of your environment:

  • Commit: 3a0db9d
  • Isaac Sim Version: 5.0
  • OS: Ubuntu 22.04
  • GPU: 4090 RTX
  • CUDA: 12.2
  • GPU Driver: 535.129.03

Additional context

The error is due to the forgotten else condition:

if "pbt" in agent_cfg:
if agent_cfg["pbt"]["directory"] == ".":
log_root_path = os.path.abspath(log_root_path)
else:
log_root_path = os.path.join(agent_cfg["pbt"]["directory"], log_root_path)

Checklist

  • I have checked that there is no similar issue in the repo (required)
  • I have checked that the issue is not in running Isaac Sim itself and is related to the repo

Acceptance Criteria

  • log_root_path is the absolute path

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions