Skip to content

Support USD cone and focus shaping on quad lights - #376

Merged
tarcila merged 1 commit into
NVIDIA:next_releasefrom
kvankooten:quad-light-shaping
Oct 20, 2026
Merged

tarcila merged 1 commit into
NVIDIA:next_releasefrom
kvankooten:quad-light-shaping

Conversation

@kvankooten

@kvankooten kvankooten commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Quad lights currently emit without angular cone or focus controls. Add ANARI_VISRTX_LIGHT_QUAD_SHAPING and ANARI_VISRTX_LIGHT_QUAD_FOCUS to expose USD-style cone angle/softness and focus/tint parameters.

Illumination and visible/reflected light proxies use the same radiance attenuation. Unset parameters preserve the existing behavior, and uniform-area sampling and its probability density remain unchanged.

This branch contains one feature commit directly on next_release, with no MDL or texture dependency.

Validation after rebasing onto next_release (73c02f6a, 2026-10-20): Windows Release rebuild with CUDA 13.2 and OptiX 7.7, with MDL disabled. TestQuadLightShaping, TestQuadLightFocus, 6 host unit tests and 4 existing lighting regressions all passed.

The combined rendering-integration branch starts from the merged upstream MDL implementation and adds both lighting branches. All 21 feature, host and rendering checks passed, including the upstream MDL descriptor/binding tests and shaping-plus-texture composition.

@jeffamstutz
jeffamstutz requested a review from tarcila October 16, 2026 16:29

@tarcila tarcila left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me. One small thing that I'll address later on: the light picker ignores cone and focus. Still unbiased, but slower to converge when the feature is used.

@tarcila
tarcila merged commit 525d93a into NVIDIA:next_release Oct 20, 2026
8 checks passed
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