Skip to content

Conversation

Kashu7100
Copy link
Collaborator

@Kashu7100 Kashu7100 commented Aug 13, 2025

Description

  • Remove rendering on reset so that adding camera won't affect the performance unless called render explicitly

Motivation and Context

Enforcing camera rendering at reset significantly impedes performance during RL training because the length of the episodes is very short at the beginning of the learning process. Moreover, it serves no purpose since it will never be displayed anyway. Historically, this was necessary to avoid some segfault as far as I know, but it is no longer the case.

How Has This Been / Can This Be Tested?

Running python examples/rigid/single_franka.py --vis and python examples/rigid/single_franka_batch_render.py -r -d

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.

@duburcqa duburcqa changed the title [BUG FIX] remove render at reset [BUG FIX] Avoid rendering cameras at reset. Aug 13, 2025
@duburcqa duburcqa changed the title [BUG FIX] Avoid rendering cameras at reset. [MISC] Avoid rendering cameras at reset. Aug 13, 2025
@duburcqa duburcqa enabled auto-merge (squash) August 13, 2025 03:09
@duburcqa duburcqa merged commit 75c5ed1 into Genesis-Embodied-AI:main Aug 13, 2025
18 checks passed
YilingQiao pushed a commit to YilingQiao/Genesis that referenced this pull request Aug 22, 2025
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.

2 participants