Skip to content

fix: materialize MuJoCo pool outside hot path - #290

Merged
TATP-233 merged 1 commit into
mainfrom
fix/mujoco-pool-lifecycle-slow-tests
Apr 16, 2026
Merged

TATP-233 merged 1 commit into
mainfrom
fix/mujoco-pool-lifecycle-slow-tests

Conversation

@TATP-233

Copy link
Copy Markdown
Collaborator

Summary

  • materialize MuJoCo BatchEnvPool explicitly after init domain randomization instead of lazy hot-path creation
  • keep init randomization before pool materialization so model variants do not require duplicate pool builds
  • make Allegro missing grasp cache fall back to procedural reset and update slow tests for the lifecycle contract

Validation

  • UV_CACHE_DIR=/tmp/uv-cache uv run pytest -m "slow and not veryslow" tests/base/test_sim_backend.py::TestMuJoCoBasic tests/base/test_sim_backend.py::TestMuJoCoBodySensors tests/base/test_sim_backend.py::TestCrossBackend tests/base/test_sim_backend.py::TestCrossBackendBodySensors tests/base/test_sim_backend.py::test_motrix_backend_fixed_base_set_state_matches_mujoco_for_hand_and_ball tests/envs/test_allegro_domain_randomization.py::test_allegro_mujoco_reset_applies_base_mass_and_com_domain_randomization 'tests/envs/test_env_configs.py::test_env_reset_and_step[AllegroInhandRotation]' tests/envs/test_env_configs.py::test_go1_env_initializes_kp_kd_into_pool tests/envs/test_env_configs.py::test_go2_env_initializes_kp_kd_into_pool tests/envs/test_env_configs.py::test_allegro_env_initializes_kp_kd_into_pool -q
  • UV_CACHE_DIR=/tmp/uv-cache make test-slow

@TATP-233
TATP-233 merged commit 2a5c4ed into main Apr 16, 2026
10 checks passed
@TATP-233
TATP-233 deleted the fix/mujoco-pool-lifecycle-slow-tests branch April 16, 2026 20:47
TATP-233 added a commit that referenced this pull request May 4, 2026
…-tests

fix: materialize MuJoCo pool outside hot path
TATP-233 added a commit that referenced this pull request May 30, 2026
…-tests

fix: materialize MuJoCo pool outside hot path
TATP-233 added a commit that referenced this pull request Sep 4, 2026
…-tests

fix: materialize MuJoCo pool outside hot path
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.

1 participant