haosulab / ManiSkill

SAPIEN Manipulation Skill Framework, a GPU parallelized robotics simulator and benchmark
https://maniskill.ai/
Apache License 2.0
920 stars 166 forks source link

[Bug] Fix: (unideal) only do .step() if not doing partial reset #672

Closed arth-shukla closed 3 weeks ago

arth-shukla commented 3 weeks ago

Doing an extra px.step() can cause a slight slowdown. If many parallel envs, partial resets are more common, and this can cause large compounding slowdowns. However, these are needed for stability in rcad scene_builders. Until partial px.step() is possible, this is an unideal fix.