Skip to content

feat: add Manager-Based root-state reset contract - #1073

Merged
TATP-233 merged 1 commit into
dev/issue-1042-manager-based-apifrom
feat/issue-1072-root-state-contract
Aug 17, 2026
Merged

feat: add Manager-Based root-state reset contract#1073
TATP-233 merged 1 commit into
dev/issue-1042-manager-based-apifrom
feat/issue-1072-root-state-contract

Conversation

@TATP-233

Copy link
Copy Markdown
Collaborator

Summary

  • add a typed BackendRootStateLayout contract for canonical xyz+wxyz qpos and world-linear/body-angular qvel columns
  • resolve and cache floating-root layouts in MuJoCo, Motrix, and MJWarp; keep Drake fail-closed because its current runtime metadata does not prove body-to-free-root ownership
  • expose read-only 13-D EntityData.default_root_state plus transaction-owned root state/pose/velocity writes
  • preserve one SimBackend.set_state commit per reset while converting community world angular velocity with the staged quaternion

Relates #1072
Umbrella: #1042

Scope and size

  • source-derived lines: 0
  • mechanical NumPy conversion lines: 0
  • UniLab-specific additions: 826 lines (implementation, diagnostics, and focused tests)
  • modified existing files: 12
  • deleted lines/files: 9 lines / 0 files
  • no event sampling term, production task migration, Hydra, runner, IPC, or script change

Validation

  • make test-all
    • ruff format/check: passed
    • mypy: passed
    • pyright: passed
    • pytest: 1949 passed, 26 skipped, 275 deselected, 1 xfailed
    • benchmark smoke: module 32/33 and script 33/34; only platform-optional MLX skipped
  • focused root/backend/entity/reset tests: 97 passed, 3 skipped, 3 deselected
  • real MuJoCo/Motrix conformance verifies body-frame generalized angular velocity reads back as the expected world-frame body velocity

@TATP-233
TATP-233 merged commit 9630680 into dev/issue-1042-manager-based-api Aug 17, 2026
7 checks passed
@TATP-233
TATP-233 deleted the feat/issue-1072-root-state-contract branch August 17, 2026 19:06
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