Skip to content

rtx: environment-map MIS (balance heuristic)#311

Merged
jeffamstutz merged 1 commit into
NVIDIA:next_releasefrom
tarcila:multiple-importance-sampling-hdri
Jun 12, 2026
Merged

rtx: environment-map MIS (balance heuristic)#311
jeffamstutz merged 1 commit into
NVIDIA:next_releasefrom
tarcila:multiple-importance-sampling-hdri

Conversation

@tarcila

@tarcila tarcila commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Add balance-heuristic multiple importance sampling for the environment light across the Quality and Interactive path tracers and MDL:

  • NextRay.pdf carries the BSDF sampling pdf for the next bounce
  • pbrBsdfPdf + view-only PBR lobe selection
  • materialEvalPdf EvaluatePdf callable
  • MDL env MIS via mdlBsdf_evaluate / sample_data.pdf

Also fix HDRI radiance being double-counted on bounced-ray misses in the Quality path tracer.

Gets better first frames for low sample counts:

render modebefore (16spp)after (16spp)reference (2048spp)
interactive abg-interactive-16spp before abg-interactive-16spp after abg-interactive-2048spp
quality abg-quality-16spp before abg-quality-16spp after abg-quality-2048spp

Add balance-heuristic multiple importance sampling for the environment
light across the Quality and Interactive path tracers and MDL:
  - NextRay.pdf carries the BSDF sampling pdf for the next bounce
  - pbrBsdfPdf + view-only PBR lobe selection
  - materialEvalPdf EvaluatePdf callable
  - MDL env MIS via mdlBsdf_evaluate / sample_data.pdf

Also fix HDRI radiance being double-counted on bounced-ray misses in the
Quality path tracer.
@tarcila tarcila requested a review from jeffamstutz June 12, 2026 14:19

@jeffamstutz jeffamstutz 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!

@jeffamstutz jeffamstutz merged commit 3282827 into NVIDIA:next_release Jun 12, 2026
8 checks passed
@tarcila tarcila deleted the multiple-importance-sampling-hdri branch June 13, 2026 19:53
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