Skip to content

Apply reused render product configuration on first execution - #783

Open
sylvesterkaczmarek wants to merge 7 commits into
isaac-sim:mainfrom
sylvesterkaczmarek:fix/reused-render-product-config
Open

Apply reused render product configuration on first execution#783
sylvesterkaczmarek wants to merge 7 commits into
isaac-sim:mainfrom
sylvesterkaczmarek:fix/reused-render-product-config

Conversation

@sylvesterkaczmarek

Copy link
Copy Markdown

Description

Ensure IsaacCreateRenderProduct applies the requested camera and resolution immediately when reusing an existing render product.

The reuse path previously returned as soon as it accepted inputs:renderProductPrim. That bypassed the common camera/resolution update path below it, so a supplied render product with stale settings could be reported as ready for one evaluation before being corrected on a later tick.

The reuse path now applies inputs:width, inputs:height, and inputs:cameraPrim before returning and synchronizes the cached state with those values.

Validation

  • extends the existing CreateRenderProduct graph test
  • supplies a 512x512 render product bound to /OmniverseKit_Persp
  • requests the default 1280x720 resolution and /OmniverseKit_Top
  • verifies both settings are applied after the first graph update
  • existing create, disable, resize, camera-switch, and reuse coverage remains in place

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