Skip to content

feat: remove retired task owners and WarpSAC - #2072

Merged
TATP-233 merged 1 commit into
mainfrom
feat/remove-retired-tasks-and-warpsac
Oct 8, 2026
Merged

TATP-233 merged 1 commit into
mainfrom
feat/remove-retired-tasks-and-warpsac

Conversation

@TATP-233

@TATP-233 TATP-233 commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Completely removed retired task owners and their implementation/config/test/doc/asset surface:
    • allegro_inhand
    • allegro_inhand_grasp
    • fr3_joint_target
    • g1_box_tracking
    • stewart_balance
    • x2_wall_flip_tracking
    • simtool_real
    • g1_wbt_obs
  • Completely removed WarpSAC:
    • deleted conf/warpsac/** and train_warpsac.py
    • removed CLI, completion, interactive playback, off-policy dispatch, structured config, support-matrix, audit, and test wiring
    • retained the existing SAC and FlashSAC paths
  • Removed task-owned assets and asset-hub registrations that were no longer referenced, including Allegro, Stewart, X2, and FR3 native assets.
  • Updated bilingual task/algorithm/deployment docs, demo documentation, API autosummary, generated support matrix, and active ADR evidence.
  • User-facing impact: the removed task names and --algo warpsac now fail as unsupported; WarpSAC checkpoints/runs have no migration path in this PR. Existing retained Go2/G1 owners and SAC/FlashSAC training behavior are unchanged.

Linked Work

  • Issue: to be linked by maintainer (no existing removal issue was found)
  • Parent roadmap (when applicable): none
  • Roadmap declared base (when applicable): not applicable
  • Milestone: none
  • Base branch: main

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:

uv run scripts/generate_support_matrix.py --write
uv run pytest tests/tasks/test_package_boundary.py tests/tasks/test_production_registry_closeout.py tests/config/test_config_system.py tests/config/test_locomotion_params.py tests/config/test_g1_motion_tracking_owners.py tests/scripts/test_support_matrix.py tests/scripts/test_audit_sim2sim_contracts.py tests/assets/test_hub.py tests/test_pull_assets.py -q
uv run pytest tests/test_cli.py tests/test_completion.py tests/envs/test_env_configs.py tests/envs/test_motion_loader.py tests/algos/test_offpolicy_double_buffer_runner.py tests/algos/test_rsl_rl_runner.py tests/base/test_superdex_backend_options.py tests/base/backend/test_mujoco_scene_context_visual.py tests/base/backend/test_drake_batch_pool.py tests/benchmark/test_replay_buffer_placement_benchmark.py -q
uv run pytest tests/tasks/test_motion_term_parity.py -q
uv run pytest tests/scripts/test_check_docs.py -q
make check
make test
make test-all
UNILAB_DOCS_SKIP_AUTODOC=1 uv run --no-project --with-requirements requirements.txt sphinx-build -b html -n source build/html

Validation outcomes:

  • Focused tests: 43 passed.
  • Broader focused test set: 122 passed, 11 skipped.
  • Docs checks: 20 passed.
  • make test: 1821 passed, 42 skipped, 488 deselected.
  • make test-all: check/type gates passed; 1821 passed, 42 skipped, 488 deselected; benchmark import smoke passed 34/34 module entries and 35/35 script entries.
  • Sphinx build: succeeded with existing warnings.

Remote CI route:

  • Base main: remote CI triggered by this PR; status to be recorded from the PR checks after it starts.

Impact

  • Backend impact: none for retained backends; SuperDex remains supported through the retained Go2 owner, while the FR3 owner/assets are removed.
  • Platform impact: both
  • Training effect expected: yes — removed tasks and WarpSAC are no longer selectable; retained task/algorithm routes are unchanged.

Artifacts

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

Checklist

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

@TATP-233

TATP-233 commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator Author

Remote CI update:

@TATP-233
TATP-233 merged commit 5ac13e7 into main Oct 8, 2026
8 checks passed
@TATP-233
TATP-233 deleted the feat/remove-retired-tasks-and-warpsac branch October 8, 2026 09:57
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