Document AgiBot teleop envs in Isaac Teleop docs#6110
Conversation
Add the two keyboard/SpaceMouse AgiBot place environments (Isaac-Place-Mug-Agibot-Left-Arm-RmpFlow-v0 and Isaac-Place-Toy2Box-Agibot-Right-Arm-RmpFlow-v0) to the Keyboard and SpaceMouse Environments reference table. Both configs expose teleop_devices (keyboard, SpaceMouse) and use RMPFlow arm control with a binary gripper, matching the documented Galbot RMPFlow tasks, but were missing from the table. (cherry picked from commit 46d0961)
Greptile SummaryThis PR backports a documentation fix from
Confidence Score: 5/5Documentation-only change that adds two table rows; all task IDs and control descriptions match the registered environments. Both added task IDs were verified against agibot/init.py — they match exactly. Arm-side labels (left/right), controller (RMPFlow), and input bindings (K / SpaceMouse left) are consistent with the existing Galbot entries that use the same robot family and control scheme. The RST table syntax is valid and the new rows slot in cleanly between the Galbot and UR10 blocks. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User opens Isaac Teleop docs] --> B[Keyboard & SpaceMouse Environments table]
B --> C[Isaac-Place-Mug-Agibot-Left-Arm-RmpFlow-v0\nLeft-arm · RMPFlow · Gripper K/SpaceMouse left]
B --> D[Isaac-Place-Toy2Box-Agibot-Right-Arm-RmpFlow-v0\nRight-arm · RMPFlow · Gripper K/SpaceMouse left]
C --> E[Registered in\nagibot/__init__.py]
D --> E
Reviews (1): Last reviewed commit: "Document AgiBot teleop envs in Isaac Tel..." | Re-trigger Greptile |
There was a problem hiding this comment.
📝 Documentation Review — PR #6110
Summary: This PR adds two AgiBot teleop environment entries to the "Keyboard and SpaceMouse Environments" table in docs/source/features/isaac_teleop.rst.
✅ Looks Good
- RST formatting is consistent with the surrounding table entries (indentation, column alignment,
list-tablesyntax all correct). - Placement is logical — the AgiBot entries sit between the Galbot and UR10 blocks.
- Content follows the established pattern: env ID, supported devices, and a brief description of arm control + gripper activation.
- pre-commit and other CI checks are passing.
💡 Minor Observations (non-blocking)
-
Alphabetical/grouped ordering: The existing table loosely groups entries by robot (Franka → Galbot → UR10 → Franka again). The new AgiBot entries are placed between Galbot and UR10, which is reasonable but could also be grouped with other "Place" tasks if the table grows. Fine as-is for now.
-
Missing
Noteblock for RMPFlow? The Galbot RMPFlow entry above (Isaac-Stack-Cube-Galbot-Right-Arm-Suction-RmpFlow-v0) includes a warning about avoiding collisions that cause controller overtuning. If the same caveat applies to the AgiBot RMPFlow environments, consider adding a similar note in a follow-up. -
PR body references
#<develop PR>with a link to #6109 — just confirming the backport chain is tracked. The cherry-pick provenance is noted in the body, which is good practice.
Verdict
Clean docs-only change, correctly formatted. No issues found. 👍
Description
Backport of # to
release/3.0.0-beta2.Cherry-picks the documentation fix that adds the two AgiBot place tasks to the
Keyboard and SpaceMouse Environments table in
docs/source/features/isaac_teleop.rst:Isaac-Place-Mug-Agibot-Left-Arm-RmpFlow-v0Isaac-Place-Toy2Box-Agibot-Right-Arm-RmpFlow-v0The original QA report was filed against the 3.0.0-beta2 docs, so the fix is
backported here as well. Both task IDs are registered on this release (at
manager_based/manipulation/place/config/agibot/— the pre-contrib/path) andexpose
teleop_devices(keyboard, SpaceMouse), so the table entries are accuratefor beta2. Applied cleanly via
git cherry-pick -x(provenance recorded).Fixes # (issue)
Type of change
Checklist
pre-commitchecks with./isaaclab.sh --formatconfig/extension.tomlfileCONTRIBUTORS.mdor my name already exists there