Skip to content

fix(rtx): apply ANARI channel-fill to image samplers#341

Merged
jeffamstutz merged 1 commit into
NVIDIA:next_releasefrom
tarcila:rtx-fix-floatN-to-float4-expansion
Jul 16, 2026
Merged

fix(rtx): apply ANARI channel-fill to image samplers#341
jeffamstutz merged 1 commit into
NVIDIA:next_releasefrom
tarcila:rtx-fix-floatN-to-float4-expansion

Conversation

@tarcila

@tarcila tarcila commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

CUDA texture fetches zero-fill channels absent from the array format, so 1/2-channel float color textures read back alpha=0.

Matte/PBR fold color alpha into opacity (opacity = color.w * opacity), so under alphaMode=blend those surfaces went fully transparent and vanished.

@tarcila
tarcila requested a review from jeffamstutz July 15, 2026 19:46
Comment thread devices/rtx/device/gpu/evalMaterialParameters.h Outdated
CUDA texture fetches zero-fill channels absent from the array format, so
1/2-channel float color textures read back alpha=0. Matte/PBR fold color
alpha into opacity (opacity = color.w * opacity), so under alphaMode=blend
those surfaces went fully transparent and vanished; helide filled alpha=1
and rendered them. Carry numChannels in SamplerGPUData (from the existing
Sampler::numChannels()) and restore the ANARI convention in evaluateSampler
(missing color -> 0, alpha -> 1), matching the attribute path. MDL texture
helpers keep MDL semantics, untouched.

Adds TestSamplerAlphaFill: blend must render == opaque within a channel
count; fails pre-fix (1-channel blend collapses to black).
@tarcila
tarcila force-pushed the rtx-fix-floatN-to-float4-expansion branch from 259c322 to 23b0c50 Compare July 15, 2026 23:17
@tarcila
tarcila requested a review from jeffamstutz July 15, 2026 23:19
@jeffamstutz
jeffamstutz merged commit 6391597 into NVIDIA:next_release Jul 16, 2026
8 checks passed
@tarcila
tarcila deleted the rtx-fix-floatN-to-float4-expansion branch July 16, 2026 16:38
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