Document AgiBot keyboard/SpaceMouse teleop environments#6109
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.
There was a problem hiding this comment.
📝 Documentation Review — PR #6109
Summary: This PR adds two AgiBot place-task 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 in isaac_teleop.rst.
✅ What Looks Good
- Correct placement: The entries are inserted adjacent to the existing Galbot entries (which also use RMPFlow), maintaining logical grouping by robot platform.
- Consistent formatting: The RST list-table syntax matches the surrounding entries (indentation, field separators, inline bold markup).
- Accurate descriptions: The arm control method (RMPFlow end-effector pose) and gripper trigger (
K/ left SpaceMouse button) are clearly documented and consistent with how similar entries describe their controls.
💡 Minor Suggestions (non-blocking)
-
Consider adding a task description prefix: Other entries in the table include a brief mention of the task objective (e.g., "Same as left-arm gripper above…"). These two entries jump straight to control descriptions. Adding a short phrase like "Place mug task." or "Place toy into box." before the arm/gripper details would help readers understand what the environment does at a glance.
-
Ordering within the table: The new AgiBot entries sit between the Galbot visuomotor entry and the UR10 entry. If there is an intended ordering convention (alphabetical by robot, or grouped by robot family), it may be worth double-checking placement. Currently it reads naturally since Galbot → AgiBot → UR10 groups bimanual/humanoid platforms together.
🔍 Verification Notes
- No code changes — documentation only (8 additions, 0 deletions in a single
.rstfile). - RST indentation appears correct (3-space continuation for multi-line descriptions within the list-table directive).
- No broken cross-references or directives introduced.
Overall: Clean, minimal, and well-scoped docs addition. LGTM with the minor readability suggestion above. 🚀
Greptile SummaryThis PR adds two missing AgiBot place-task environments to the keyboard/SpaceMouse teleop reference table in
Confidence Score: 5/5Documentation-only change; both environment IDs exist in the task registry with matching arm sides and teleop device configurations. Both added environment IDs are confirmed present in the task registry. The left/right arm designations match the actual joint configuration in the respective env config files. Both configs have teleop_devices set up with keyboard and SpaceMouse as described. The table placement is consistent with existing entries. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[isaac_teleop.rst Table] --> B[Galbot entries]
A --> C[Agibot entries - NEW]
A --> D[UR10 entries]
A --> E[Franka entry]
C --> C1["Isaac-Place-Mug-Agibot-Left-Arm-RmpFlow-v0\nDevices: Keyboard, SpaceMouse\nArm: left-arm RMPFlow | Gripper: K / left btn"]
C --> C2["Isaac-Place-Toy2Box-Agibot-Right-Arm-RmpFlow-v0\nDevices: Keyboard, SpaceMouse\nArm: right-arm RMPFlow | Gripper: K / left btn"]
Reviews (1): Last reviewed commit: "Document AgiBot teleop envs in Isaac Tel..." | Re-trigger Greptile |
Description
Adds the two AgiBot place tasks to the Keyboard and SpaceMouse Environments
reference table in
docs/source/features/isaac_teleop.rst:Isaac-Place-Mug-Agibot-Left-Arm-RmpFlow-v0Isaac-Place-Toy2Box-Agibot-Right-Arm-RmpFlow-v0Both environments expose
teleop_devices(keyboard, SpaceMouse) and use RMPFlowarm control with a binary gripper, but were absent from the table even though
their Galbot/UR10 peers are listed. They use the legacy
isaaclab.devicesstack,so they belong
Fixes # (issue)
Type of change
Checklist
pre-commitchecks with./isaaclab.sh --formatconfig/extension.tomlfileCONTRIBUTORS.mdor my name already exists there