Skip to content

Fix: Add headless config to AppLauncher in export_env_for_envhub#35

Open
yjmade wants to merge 1 commit intomainfrom
ai-fix/issue-29
Open

Fix: Add headless config to AppLauncher in export_env_for_envhub#35
yjmade wants to merge 1 commit intomainfrom
ai-fix/issue-29

Conversation

@yjmade
Copy link
Collaborator

@yjmade yjmade commented Mar 3, 2026

Fixes #29

Problem

In export_env_for_envhub, AppLauncher is initialized without the headless parameter, causing cfg.headless to be implicitly overwritten by AppLauncher defaults. This results in errors when running on headless servers:

RuntimeError: Cannot render 'rgb_array' when the simulation render mode is 'NO_GUI_OR_RENDERING'

Solution

  1. Add headless: True and enable_cameras: False to default config in _parse_config
  2. Pass headless=cfg.headless to AppLauncher initialization

Changes

  • lw_benchhub/utils/envhub_utils.py: Updated export_env_for_envhub function

Testing

On a headless server, running export_env_for_envhub should no longer raise RuntimeError.

Closes #29

- Add 'headless' and 'enable_cameras' to default config in _parse_config
- Pass headless=cfg.headless to AppLauncher to prevent implicit override
- Fixes RuntimeError when running on headless servers with NO_GUI_OR_RENDERING
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

Successfully merging this pull request may close these issues.

cfg.headless is being implicitly overwritten by AppLauncher defaults in export_env_for_envhub

1 participant