Skip to content

restore: Bumps client back to 6.2.0 for CXR.js#555

Open
yanziz-nvidia wants to merge 1 commit into
mainfrom
yanziz/revert-revert-620
Open

restore: Bumps client back to 6.2.0 for CXR.js#555
yanziz-nvidia wants to merge 1 commit into
mainfrom
yanziz/revert-revert-620

Conversation

@yanziz-nvidia
Copy link
Copy Markdown
Contributor

@yanziz-nvidia yanziz-nvidia commented May 22, 2026

This reverts commit 3a22354.

Summary by CodeRabbit

  • New Features

    • Added real-time performance metrics visualization, displaying Render FPS, Streaming FPS, and Pose-to-Render latency information.
  • Chores

    • Updated CloudXR Web SDK to version 6.2.0.
    • Updated supporting dependencies to latest compatible versions.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 5d02ff74-f802-4dce-b602-b2703ac13cd2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

This PR updates the CloudXR SDK from 6.1.0 to 6.2.0 across environment configuration, package manifest, and telemetry reporting. It introduces a new PerformanceCanvasImage React component that renders real-time performance metrics (Render FPS, Streaming FPS, and Pose-to-Render latency) onto an offscreen HTML canvas, converts it to a three.js texture, and displays it via a uikit Image. The UI then imports and integrates this component, replacing the previous inline multi-block metric display with the single canvas-based component, passing through the existing signal-based text props.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • NVIDIA/IsaacTeleop#530: Contains the same file-level changes (SDK bump to 6.2.0, PerformanceCanvasImage addition, CloudXRUI refactoring).
  • NVIDIA/IsaacTeleop#552: Directly reverts the changes in this PR by downgrading SDK back to 6.1.0 and removing PerformanceCanvasImage usage.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: bumping CloudXR client from 6.1.0 to 6.2.0, which is reflected across all modified files including dependencies, version constants, and configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch yanziz/revert-revert-620

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant