Skip to content

Fix/houdini22 interactive render settings - #17

Open
gripeyes wants to merge 6 commits into
OpenMoonRay:mainfrom
gripeyes:fix/houdini22-interactive-render-settings
Open

Fix/houdini22 interactive render settings#17
gripeyes wants to merge 6 commits into
OpenMoonRay:mainfrom
gripeyes:fix/houdini22-interactive-render-settings

Conversation

@gripeyes

Copy link
Copy Markdown

Compatibility:

patch

Issues/Tickets:

OpenMoonRay/openmoonray#276
#14

Release notes comment:

Fixed interactive render-settings updates in the Houdini 22 Solaris viewport.

Comments for the reviewer:

This PR is stacked directly on the head of #14 (85e63fc), Rob’s hdm_10 branch. Once #14 merges into main, this PR will contain only the interactive render-settings commit.

Houdini 22 identifies Solaris viewport renders with the houdini:viewport setting, while older versions used houdini:interactive. An active RenderSettings prim is also present during viewport rendering, so its presence alone cannot select the one-shot product-render path.

This change:

  • recognizes both houdini:viewport and houdini:interactive
  • applies client render settings before choosing the render path
  • keeps Solaris viewport rendering on the interactive render-pass path
  • tracks RenderSettings prim synchronization versions
  • clears product-render completion when settings, products or renderer updates change
  • preserves the one-shot path for actual offline product renders

Unrelated trailing whitespace changes from the validation commit were removed.

Umbrella validation PR:
OpenMoonRay/openmoonray#276

Houdini 22 build notes:
https://github.com/gripeyes/openmoonray/blob/test/houdini22-clean-build/building/macOS/Houdini22_build_notes.md

Look or scene setup change:

No expected look change. Render-setting edits now propagate correctly during interactive Solaris rendering.

Special notes for production:

Depends on the HdMoonray 10/Hydra 2.0 work in #14.

Attention/Reviewers:

@robwilson-dwa

AI Assisted Development:

Assisted-by: OpenAI Codex / GPT-5.6

Checklist:

  • Documentation has been updated.
  • Includes new unit tests.
  • Includes new RATS tests.

robwilson-dwa and others added 6 commits August 21, 2026 15:32
* WIP

* light instancing first version

* fix tests, update packages

* CR fixes

* another CR fix

* material/light linking

* move hats to usdrecord

* support hats tests in 2.0 mode

* renable subset light linking

* create MoonrayScene

* factor out Options from RenderDelegate

* fix and test MoonrayMeshLight

* tidy up class names

* clean up tokens

* motion blur changes

* minor fixes

* add back default light

* inputs: allowed on custom attrs

* version to 10.0

* added simplify paths option for testing

* fix crash on exit

* add RenderSettings support

* remove stray debug print

* HDM-408 : changes from OMR PR#8

* fix instancing issue, add option to limit geom tess

* fix inst selection, remove nested inst selection

* add back nested inst selection

* [skip ci] Bumped minor version for the release.

* factor out scene_rdl2 specifics

* clean up render outputs

* scene_rdl2 cleanup

* [skip ci] Bumped minor version for the release.

* fix compile error for usd 0.25.11

* [skip ci] Bumped minor version for the release.

* terminal proxies (disabled)

* disable build for usd<0.25.5

---------

Co-authored-by: Rob Wilson <rwilson@dreamworks.com>
Co-authored-by: 206531218_nbcuni <toshi.kato@dreamworks.com>
Co-authored-by: 206676206_nbcuni <Dan.McCann@dreamworks.com>
Co-authored-by: 206689199_nbcuni <Ashley.Lee@dreamworks.com>
Signed-off-by: Rob Wilson <rwilson@dreamworks.com>
Signed-off-by: Rob Wilson <rob.wilson@dreamworks.com>
Signed-off-by: Rob Wilson <rwilson@dreamworks.com>
Signed-off-by: Rob Wilson <rob.wilson@dreamworks.com>
Signed-off-by: Rob Wilson <rwilson@dreamworks.com>
Signed-off-by: Rob Wilson <rob.wilson@dreamworks.com>
Signed-off-by: Rob Wilson <rwilson@dreamworks.com>
Signed-off-by: Rob Wilson <rob.wilson@dreamworks.com>
Signed-off-by: Rob Wilson <rwilson@dreamworks.com>
Signed-off-by: Rob Wilson <rob.wilson@dreamworks.com>
Signed-off-by: Jakub Svoboda <132791205+rolledhand@users.noreply.github.com>
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