Skip to content

feat!: move Unitree production tasks downstream - #1583

Merged
TATP-233 merged 1 commit into
mainfrom
feat/unitree-task-extraction
Sep 15, 2026
Merged

TATP-233 merged 1 commit into
mainfrom
feat/unitree-task-extraction

Conversation

@TATP-233

@TATP-233 TATP-233 commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move the requested Unitree production task families out of UniLab and into the downstream unitree_rl_unilab package.
  • Remove their UniLab task modules, registry entries, owner YAML, task-specific tests, robot metadata, migration/support-matrix claims, benchmark defaults, drift evidence, and documentation claims.
  • Retain UniLab's Manager-Based runtime, common cross-robot terms, backend/training contracts, and the retained G1 reference paths.
  • Update bilingual docs and the generated support matrix to use retained UniLab tasks first and record downstream ownership only as an ecosystem boundary.
  • Downstream implementation, all 95 owner/base YAML files, direct robot assets, tests, packaging, and public-PyPI-only upstream dependency policy are committed directly on unilabsim/unitree_rl_unilab main (migration b975322, CI parser fix 686f337).

This is a breaking task-name migration: downstream tasks use Unitree* registry names. Existing checkpoints require an explicit old run-directory selection.

Linked Work

Validation

  • make test-all passed on the final local head before this PR was created or updated
  • Additional task-specific validation listed below

Commands actually run:

make check
make test
make test-all
uv run pytest tests/scripts/test_check_docs.py -q
cd docs/sphinx && UNILAB_DOCS_SKIP_AUTODOC=1 uv run --no-project --with-requirements requirements.txt sphinx-build -b html -n source build/html

Final-head make test-all result: 1479 passed, 26 skipped, 657 deselected; benchmark import smoke passed with only the platform-optional MLX entry skipped.

Downstream local validation on unitree_rl_unilab@b975322, followed by a fresh mypy check on 686f337:

uv run --project ../UniLab pytest tests -q
uv run --project ../UniLab ruff check src tests
uv run --project ../UniLab ruff format --check src tests
uv run --project ../UniLab mypy --config-file pyproject.toml src
uv lock --check
uv build

Downstream result: 50 tests passed; lint/format/type/lock/build passed. MuJoCo loaded A2, G1, Go1, Go2, and Go2W directly packaged models and stepped 23-DoF G1 walk/motion owners. Current-main remote CI at 686f337 completed successfully: https://github.com/unilabsim/unitree_rl_unilab/actions/runs/34951091045.

Remote CI route:

Impact

  • Backend impact: mujoco / motrix
  • Platform impact: Linux
  • Training effect expected: yes — migrated task IDs are removed from UniLab and replaced by downstream Unitree* IDs

Artifacts

  • W&B: N/A
  • benchmark result: N/A
  • video / screenshot: N/A
  • ONNX / checkpoint: N/A

Checklist

  • Added or updated tests where needed
  • Updated docs if behavior or workflow changed
  • Linked the driving issue
  • Noted any follow-up work explicitly

Follow-up: downstream license provenance should be documented for the directly committed robot meshes/textures.

@TATP-233
TATP-233 requested a review from caozx1110 as a code owner September 15, 2026 09:01
@TATP-233
TATP-233 merged commit c7f2c87 into main Sep 15, 2026
8 checks passed
@TATP-233
TATP-233 deleted the feat/unitree-task-extraction branch September 15, 2026 09:11
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