Skip to content

feat(env): add Manager-Based Go2 gait phase terms - #1083

Merged
TATP-233 merged 1 commit into
dev/issue-1042-manager-based-apifrom
feat/issue-1082-go2-gait-phase-terms
Aug 18, 2026
Merged

feat(env): add Manager-Based Go2 gait phase terms#1083
TATP-233 merged 1 commit into
dev/issue-1042-manager-based-apifrom
feat/issue-1082-go2-gait-phase-terms

Conversation

@TATP-233

@TATP-233 TATP-233 commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add task-owned NumPy quadruped_gait_phase, preserving the legacy global float32 gait clock and four-foot ordering
  • add cached-sensor-view contact and swing-height reward terms with construction-time capability validation
  • fail closed on invalid configuration, sensor layout, backward counters, and non-finite runtime data
  • keep production Go2 wiring, core manager/backend contracts, runner, IPC, and Hydra configuration unchanged

Scope accounting

  • files: 2
  • additions: 400
  • deletions: 0
  • source-derived: 0
  • mechanical: 0
  • task-owned code: 234
  • tests: 166

Validation

  • focused tests: 8 passed
  • adjacent manager regression: 35 passed
  • ruff: passed
  • mypy: passed
  • pyright: passed
  • make test-all: passed (1989 passed, 27 skipped, 276 deselected, 1 xfailed; benchmark module mode 32/33 and script mode 33/34, with one platform-optional MLX skip in each mode)

Refs #1042
Closes #1082

#1042 child policy: all validation above ran locally on the final head. Remote CI/Docs workflows are intentionally not required for child PRs.

@TATP-233
TATP-233 merged commit 247bc30 into dev/issue-1042-manager-based-api Aug 18, 2026
5 of 7 checks passed
@TATP-233
TATP-233 deleted the feat/issue-1082-go2-gait-phase-terms branch August 18, 2026 10:42
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