Archived: remove camera_teleop + dead arm_hand.py - #266
Merged
Conversation
Superseded by the mjlab badminton environment (#179) — a full rewrite on the simulation_mj stack with a v2 arm + racket, court, and aerodynamics. The Isaac version was the earlier attempt and its only robot config (arm_hand.ARM_CFG) points at a USD that was deleted in #183. `arm_hand.py` / `ARM_CFG` is now used only by src/teleop/camera_teleop/, which is being archived in a follow-up. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EtusD8QWFBLpdMUZPpye5B
camera_teleop (webcam hand landmarks -> DexRetargeting -> single-arm sim) was "not wired yet" and is unmaintained. Removing it retires its only dependency, pioneer_humanoid/arm_hand.py (ARM_CFG / LEFT_ARM_CFG) — a single-right-arm+hand config whose USD was deleted in #183. Everything else is on BIMANUAL_ARM_CFG. Stacked on the Isaac-badminton archival (the other ARM_CFG consumer). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EtusD8QWFBLpdMUZPpye5B
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EtusD8QWFBLpdMUZPpye5B
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #265.
camera_teleop/(webcam hand landmarks → DexRetargeting → single-arm sim) is marked "not wired yet" in the teleop README and is unmaintained. Removing it retires its only dependency:pioneer_humanoid/arm_hand.py(ARM_CFG/LEFT_ARM_CFG) — the legacy single-right-arm+hand config whose USD was deleted in #183. All live code is onBIMANUAL_ARM_CFG.src/teleop/camera_teleop/(6 files) removedsrc/pioneer_humanoid/pioneer_humanoid/arm_hand.pyremovedMerge #265 first (or merge this into it).