Skip to content

Support linear emission textures on quad lights - #377

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

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

Conversation

@kvankooten

@kvankooten kvankooten commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Quad lights currently support only constant emission color. Add ANARI_VISRTX_LIGHT_QUAD_TEXTURE: an optional colorMap array of finite, nonnegative linear RGB values multiplies the light's emitted radiance.

The same bilinearly filtered image is used for illumination and visible/reflected emission. Image edits refresh the CUDA texture and mean used for light-selection power; removing or replacing the map preserves array/texture ownership. The implementation uses the current array-owned CUDA residency API.

This branch contains one texture feature commit directly on next_release, after the MDL and quad-light shaping PRs were merged upstream. Textures multiply the existing cone/focus attenuation in both light sampling and visible/reflected emission. The texture test checks this composition.

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

The rendering-integration branch now starts from upstream's merged MDL and shaping implementations and adds only the remaining texture branch. Its tracked source tree is identical to the MDL-enabled version built and validated earlier on 2026-10-20: all 21 feature, host and rendering checks passed, including the upstream MDL descriptor/binding tests and shaping-plus-texture composition. Those MDL-enabled results were reused based on exact source-tree equality; they were not rerun for this history-only integration update.

@tarcila

tarcila commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

@kvankooten Merging #376 created conflicts with PR. Can you update it? Thanks!

@kvankooten

Copy link
Copy Markdown
Collaborator Author

That was expected, should be all good now!

@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.

LGTM, thanks!

@tarcila
tarcila merged commit 0e79325 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