Skip to content

Correct the CloudXR config for Quest handtracking#605

Merged
qianl-nv merged 1 commit intomainfrom
qianl/fix_teleop_doc
Apr 15, 2026
Merged

Correct the CloudXR config for Quest handtracking#605
qianl-nv merged 1 commit intomainfrom
qianl/fix_teleop_doc

Conversation

@qianl-nv
Copy link
Copy Markdown
Collaborator

Summary

The config in the doc is mistakenly set for AVP instead of for Quest/Pico handtracking. Correcting the doc.
This fixes issue reported from https://nvbugspro.nvidia.com/bug/6076546

@qianl-nv qianl-nv marked this pull request as ready for review April 15, 2026 09:48
@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Apr 15, 2026

Greptile Summary

This PR corrects a documentation error where the CloudXR hand-tracking config for Meta Quest 3 / Pico 4 Ultra was mistakenly using the Apple Vision Pro config (NV_DEVICE_PROFILE=auto-native + NV_CXR_ENABLE_PUSH_DEVICES=0) instead of just NV_CXR_ENABLE_PUSH_DEVICES=0. Both static_manipulation and sequential_static_manipulation workflow docs are updated consistently. The locomanipulation workflow is unaffected as it uses controllers rather than hand tracking and does not include this config block.

Confidence Score: 5/5

Documentation-only fix correcting a device-specific config error; safe to merge.

Both files are updated consistently, no code logic is changed, the locomanipulation doc (which lacks this config block) is correctly left untouched, and no P0/P1 issues were found.

No files require special attention.

Important Files Changed

Filename Overview
docs/pages/example_workflows/sequential_static_manipulation/step_2_teleoperation.rst Quest/Pico hand-tracking CloudXR config corrected to use only NV_CXR_ENABLE_PUSH_DEVICES=0, removing the AVP-specific NV_DEVICE_PROFILE=auto-native setting.
docs/pages/example_workflows/static_manipulation/step_2_teleoperation.rst Same Quest/Pico hand-tracking CloudXR config correction applied as in the sequential_static_manipulation counterpart — both docs now consistent.

Sequence Diagram

sequenceDiagram
    participant User
    participant QuestPico as Meta Quest 3 / Pico 4 Ultra
    participant AVP as Apple Vision Pro
    participant CloudXR as CloudXR Runtime

    Note over User,CloudXR: Quest/Pico Hand Tracking Setup (corrected)
    User->>CloudXR: echo NV_CXR_ENABLE_PUSH_DEVICES=0 > handtracking.env
    User->>CloudXR: python -m isaacteleop.cloudxr --cloudxr-env-config=handtracking.env
    QuestPico->>CloudXR: Open browser → https://nvidia.github.io/IsaacTeleop/client
    QuestPico->>CloudXR: Connect (hand tracking active)

    Note over User,CloudXR: AVP Hand Tracking Setup (unchanged)
    User->>CloudXR: printf NV_DEVICE_PROFILE=auto-native + NV_CXR_ENABLE_PUSH_DEVICES=0 > avp.env
    User->>CloudXR: python -m isaacteleop.cloudxr --cloudxr-env-config=avp.env
    AVP->>CloudXR: Launch Isaac XR Teleop app → Connect
Loading

Reviews (1): Last reviewed commit: "Correct the CloudXR config for Quest han..." | Re-trigger Greptile

Copy link
Copy Markdown
Collaborator

@alexmillane alexmillane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this!

@qianl-nv qianl-nv merged commit 87ca851 into main Apr 15, 2026
9 of 10 checks passed
@qianl-nv qianl-nv deleted the qianl/fix_teleop_doc branch April 15, 2026 11:10
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.

2 participants