Skip to content

Upgrade OVRTX to 0.5, OVStage to 0.2, and OvPhysX to 0.6.3 - #7861

Open
nvsekkin wants to merge 9 commits into
isaac-sim:developfrom
nvsekkin:esekkin/upgrade-ovrtx-050
Open

nvsekkin wants to merge 9 commits into
isaac-sim:developfrom
nvsekkin:esekkin/upgrade-ovrtx-050

Conversation

@nvsekkin

@nvsekkin nvsekkin commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Description

Upgrades the optional OV dependency set to:

  • ovrtx==0.5.0.377615
  • ovstage==0.2.0.377349
  • ovphysx==0.6.3
  • omniverseclient==2.74.0

All packages resolve from public PyPI. These versions form one compatibility set and must be upgraded together.

The change also updates the OvPhysX installation documentation, compatibility metadata, deformable rendering references, and CPU-resident selectors required by CPU-backed OvPhysX properties.

Migration

uv sync --inexact --extra ov

Use --extra ovrtx or --extra ovphysx when only one runtime is needed. CPU-backed OvPhysX property writes require CPU-resident values and selectors.

Known test state:

  • OVStage color parity and the Kuka RTX Minimal black-surface case remain under investigation.

Neither case is masked or rebaselined by this PR.

Type of change

  • Breaking change
  • Documentation update

Release backport

  • Backport this pull request to the active release branch after it merges into develop

Checklist

  • I have read and understood the contribution guidelines
  • I have run the pre-commit checks with ./isaaclab.sh --format
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have added a changelog fragment under source/<pkg>/changelog.d/ for every touched package (do not edit CHANGELOG.rst or bump extension.toml — CI handles that)
  • I have added my name to the CONTRIBUTORS.md or my name already exists there

@github-actions github-actions Bot added documentation Improvements or additions to documentation isaac-lab Related to Isaac Lab team infrastructure labels Sep 17, 2026
@nvsekkin nvsekkin changed the title Upgrade ovrtx-0.5.0/ovstage-0.2.0/ovphysx-0.6.3 Upgrade OVRTX to 0.5, OVStage to 0.2, and OvPhysX to 0.6.3 Sep 17, 2026
@nvsekkin

Copy link
Copy Markdown
Collaborator Author

run-ci

@isaaclab-bot isaaclab-bot Bot added ci:run-docker Trigger the on-demand Docker and GPU CI workflow and removed ci:run-docker Trigger the on-demand Docker and GPU CI workflow labels Sep 18, 2026

@isaaclab-review-bot isaaclab-review-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Isaac Lab Review Bot

Reviewed the coordinated OV compatibility-set upgrade to OVRTX 0.5.0.377615, OVStage 0.2.0.377349, OvPhysX 0.6.3, and omniverseclient 2.74.0. The dependency pins, compatibility metadata, installation guidance, CPU-resident selectors, and renderer golden references are updated consistently; no actionable candidate findings were supplied.

  • Design and architecture: The existing version-based compatibility architecture remains intact: runtime behavior is selected from installed package metadata at established OVStage 0.2, OvPhysX 0.6, and OVRTX 0.5 boundaries. The root version table, optional extras, and isaaclab_ov package dependency reflect the coordinated compatibility set.
  • API: The dependency upgrade is explicitly marked as breaking and includes migration guidance. The OV extras, package-level OVStage requirement, OVRTX installation error, and OvPhysX documentation use the updated versions consistently, without removing or renaming public symbols.
  • Implementation: The pin-consistency test was updated, CPU-backed deformable material selectors are explicitly allocated on CPU, and compatibility-module documentation matches the existing fallback paths. The regenerated Franka renderer baselines carry some review risk because binary image differences are represented only by new LFS hashes, particularly given the noted unresolved color-parity investigation, but the patch provides no concrete failure demonstrating that these baselines are incorrect.

No blocking issues. No inline issue met the actionable-evidence threshold; the assessment above records the review feedback.

Automated review; human maintainers own approval decisions.

@greptile-apps

greptile-apps Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the coordinated dependency pins, supported-platform lock resolution, compatibility metadata, and CPU-resident selector changes are internally consistent.

Summary

This PR upgrades the optional OV runtime compatibility set and updates the associated installation metadata, compatibility documentation, deformable-property selectors, and rendering baselines.

  • Pins OvPhysX 0.6.3, OVRTX 0.5.0.377615, OVStage 0.2.0.377349, and Omniverse Client 2.74.0 across project metadata and the lockfile.
  • Keeps the individual ovphysx and ovrtx extras paired with the matching OVStage release.
  • Makes deformable-material selectors explicitly CPU-resident for the new OvPhysX behavior.
  • Refreshes OVRTX RGB/RGBA baselines for the Franka cable, cloth, and soft tasks.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[ov extra] --> P[OvPhysX 0.6.3]
  A --> R[OVRTX 0.5.0.377615]
  A --> S[OVStage 0.2.0.377349]
  B[ovphysx extra] --> P
  B --> S
  C[ovrtx extra] --> R
  C --> S
  P --> D[0.6 lifecycle and CPU-backed properties]
  R --> E[Prim-path render-var keys]
  S --> F[GPU incremental hierarchy]
Loading

Reviews (1) · Last reviewed commit: "chore: Refresh lockfile after develop me..."

@nvsekkin

Copy link
Copy Markdown
Collaborator Author

run-ci

1 similar comment
@nvsekkin

Copy link
Copy Markdown
Collaborator Author

run-ci

@isaaclab-bot isaaclab-bot Bot added ci:run-docker Trigger the on-demand Docker and GPU CI workflow and removed ci:run-docker Trigger the on-demand Docker and GPU CI workflow labels Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation infrastructure isaac-lab Related to Isaac Lab team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant