Manus Documentation update for isaaclab - #6803
Conversation
Greptile SummaryThe PR updates the Manus teleoperation documentation.
Confidence Score: 4/5The documentation should not merge until the Manus launch command establishes the IsaacTeleop build directory or uses an unambiguous executable path. The documented relative plugin path is resolved from the user's current directory, while the executable is installed under a separate IsaacTeleop checkout, causing the recommended workflow to fail from the IsaacLab workspace. Files Needing Attention: docs/source/how-to/cloudxr_teleoperation.rst Important Files Changed
Reviews (1): Last reviewed commit: "Updated setup documentation to use Manus..." | Re-trigger Greptile |
There was a problem hiding this comment.
Isaac Lab Review Bot
Documentation-only update to the CloudXR/Manus guide. The newly added manual plugin-launch workflow is unclear about its relationship to automatic activation and uses a relative executable path without establishing the required working directory.
- Design and architecture: No source, package, or module architecture is affected.
- API: No public API symbols, CLI arguments, or configuration contracts are changed.
- Implementation: The manual Manus plugin step needs clarification and an unambiguous invocation path so users can reliably follow the documented workflow.
Minor fixes needed. Posted 1 actionable finding inline.
Conservative automated review; human maintainers own approval decisions.
|
Thanks @Aniket-Manus for the PR, could address the failures in the CI? :) |
Manus Documentation update for isaaclab