Resampled Importance Sampling bxdfs by devshgraphicsprogramming · Pull Request #1027 · Devsh-Graphics-Programming/Nabla

devshgraphicsprogramming · 2026-03-20T11:08:54Z

Description

Testing

TODO list:

…ession reciprocation (swapping of interface order) Unfortunately I realized our THINDIELECTRIC correction is badly designed

…a, won't implement right now

… mitsuba_xml_2_material_frontend

…ff last

@Themperror

Point Boost to the exact Wave one-line backport for emitted pragma newlines, remove the temporary local pragma workaround, and keep the remaining include-path fixes in Nabla. This leaves the Wave pragma issue fixed at the dependency level while preserving the Nabla-side fixes for Windows backslash includes and single-leading-slash virtual includes. Thanks to @Themperror for the additional pragma and include repros. Those made it straightforward to verify the dependency-level fix and drop the local workaround cleanly.

…dates Fix remaining preprocess bugs and backport Wave pragma fix

…te-nsc-windows-x64-release-master CI: Promote NSC channel to 66da590

…cet checks cache with concept

devshgraphicsprogramming · 2026-03-24T15:20:22Z

include/nbl/builtin/hlsl/bxdf/base/lambertian.hlsl

+    struct Cache {};
+    using isocache_type = Cache;
+    using anisocache_type = Cache;
+    using evalcache_type = Cache;


there should be no evalcache_type, its still same cache as for quotient and generate

But having an evalAndWeight with an overload that takes it should be a Cook Torrance / Microfacet thing only

devshgraphicsprogramming · 2026-03-24T15:21:40Z

include/nbl/builtin/hlsl/bxdf/common.hlsl

    ((NBL_CONCEPT_REQ_TYPE)(T::anisotropic_interaction_type))
    ((NBL_CONCEPT_REQ_TYPE)(T::sample_type))
+    ((NBL_CONCEPT_REQ_TYPE)(T::anisocache_type))
+    ((NBL_CONCEPT_REQ_TYPE)(T::evalcache_type))


don't require a separate evaluation cache type

devshgraphicsprogramming · 2026-03-24T15:22:45Z

include/nbl/builtin/hlsl/bxdf/common.hlsl

    ((NBL_CONCEPT_REQ_TYPE)(T::isotropic_interaction_type))
    ((NBL_CONCEPT_REQ_TYPE)(T::isocache_type))
-    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((bxdf.eval_and_weight(_sample, iso, isocache)), ::nbl::hlsl::is_same_v, typename T::quotient_pdf_type))
+    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((bxdf.evalAndWeight(_sample, iso, evalcache)), ::nbl::hlsl::is_same_v, typename T::quotient_pdf_type))


evalAndWeight taking the anisocache or isocache should be an extra overload in the imcrofacet BxDF concepts

…l methods

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl

devshgraphicsprogramming · 2026-03-25T12:55:46Z

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl

-        }
+        using g2g1_query_type = typename N::g2g1_query_type;
+        g2g1_query_type gq = ndf.template createG2G1Query<sample_type, Interaction>(_sample, interaction);
+        scalar_type G2_over_G1 = ndf.template G2_over_G1<sample_type, Interaction, MicrofacetCache>(gq, _sample, interaction, cache);


check how well this behaves as roughness ->0 we don't want to have NaNs

We have a check in ndfs for a<float_min

we're bypassing a bunch of things , so there's no check

Nabla/include/nbl/builtin/hlsl/bxdf/ndf/ggx.hlsl

Line 364 in 2539522

scalar_type G2_over_G1(NBL_CONST_REF_ARG(g2g1_query_type) query, NBL_CONST_REF_ARG(LS) _sample, NBL_CONST_REF_ARG(Interaction) interaction, NBL_CONST_REF_ARG(MicrofacetCache) cache) NBL_CONST_MEMBER_FUNC

you will in-fact assert in the constructor

Nabla/include/nbl/builtin/hlsl/bxdf/ndf/ggx.hlsl

Line 236 in 2539522

enable_if_t<C::value && !IsAnisotropic, g2g1_query_type> createG2G1Query(NBL_CONST_REF_ARG(LS) _sample, NBL_CONST_REF_ARG(Interaction) interaction) NBL_CONST_MEMBER_FUNC

of the query

Nabla/include/nbl/builtin/hlsl/bxdf/ndf/ggx.hlsl

Line 104 in 2539522

assert(a2 >= numeric_limits<scalar_type>::min);

I mean the HLSL compiles to C++ and we have a unit test so you can go through it with a debugger I guess 🙃

I think if you know/check the PDF not being INF before setting G2_over_G1 you can guard against that

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl

…xisting cache type too

devshgraphicsprogramming · 2026-03-26T09:41:59Z

include/nbl/builtin/hlsl/bxdf/common.hlsl

+#define NBL_CONCEPT_PARAM_3 (evalcache, typename T::evalcache_type)
+NBL_CONCEPT_BEGIN(4)
+#define bxdf NBL_CONCEPT_PARAM_T NBL_CONCEPT_PARAM_0
+#define _sample NBL_CONCEPT_PARAM_T NBL_CONCEPT_PARAM_1
+#define aniso NBL_CONCEPT_PARAM_T NBL_CONCEPT_PARAM_2
+#define evalcache NBL_CONCEPT_PARAM_T NBL_CONCEPT_PARAM_3
+NBL_CONCEPT_END(
+    ((NBL_CONCEPT_REQ_TYPE)(T::evalcache_type))
+    ((NBL_CONCEPT_REQ_TYPE_ALIAS_CONCEPT)(bxdf_common, T))
+    ((NBL_CONCEPT_REQ_TYPE_ALIAS_CONCEPT)(AnisotropicMicrofacetCache, typename T::anisocache_type))
+    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((bxdf.evalAndWeight(_sample, aniso, evalcache)), ::nbl::hlsl::is_same_v, typename T::value_weight_type))


cool but evalcache needs to be just microfacetCache and be of type T::cache_type

no evalcache_type

devshgraphicsprogramming · 2026-03-26T09:43:22Z

include/nbl/builtin/hlsl/path_tracing/concepts.hlsl

    ((NBL_CONCEPT_REQ_TYPE)(T::object_handle_type))
    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((sqr.getSample()), ::nbl::hlsl::is_same_v, typename T::sample_type))
-    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((sqr.getQuotientPdf()), ::nbl::hlsl::is_same_v, typename T::quotient_pdf_type))
+    ((NBL_CONCEPT_REQ_EXPR_RET_TYPE)((sqr.getQuotientPdf()), ::nbl::hlsl::is_same_v, typename T::quotient_weight_type))


getQuotientPdf rename to getQuotientAndWeight

…tes cache

keptsecret and others added 30 commits March 10, 2026 16:26

fix spirv qualifiers for HitTriangleVertexPositionsKHR

8e7615d

fix transpose when matrix N!=M

b9ec10e

fix pseudoInverse3x4, some bugs in linalg/basic.hlsl

cb13a6b

fixes lookAt calculation for loading toWorld property in mitsuba scene

c545a59

remove obsolete comments

1e907f4

Merge branch 'master' into rt_pipeline_debug_render_fixes

70b31d4

latest example

1409481

Fix checking valid jpeg file logic

5def0b4

get opacity going!

f93f430

really struggling with TWO_SIDED

1a7bf02

add CFrontendIR node copying, stack reversal and individual BxDF expr…

0dc31aa

…ession reciprocation (swapping of interface order) Unfortunately I realized our THINDIELECTRIC correction is badly designed

Beer node needs a thickness unfortunately.

8aff0f7

quick hack around normalmaps and bumpmaps, sketch out the general ide…

1b0b6f3

…a, won't implement right now

Merge remote-tracking branch 'remotes/origin/hotfix_jpeg_loader' into…

6a4e8df

… mitsuba_xml_2_material_frontend

move UV transform to front of SParameterSet, keep variable length stu…

0c1e045

…ff last

implement BLEND_BSDF

850536c

merge master, fix conflicts

9574030

I forgot that for blends all layers need to be blended!

011e7f0

I don't want to admit how long it took me to debug this one

2b34047

fix the last thing in the material frontend

2d3719f

fix TLAS export when Polygon Geometry can't be made into a BLAS

ad0cf8d

merge master

5d2568b

Merge branch 'master' into rt_pipeline_debug_render_fixes

546ed90

Fix Wave include paths and failure dumps

c86f8e1

Clean up preprocess failure plumbing

4871e24

Merge pull request #1025 from Devsh-Graphics-Programming/preprocessUp…

e4df2b2

…dates Fix remaining preprocess bugs and backport Wave pragma fix

Use upstream gh-pages action for badges

66da590

Promote NSC channel 66da590

f830a47

Merge pull request #1026 from Devsh-Graphics-Programming/ci/nsc-promo…

00017ad

…te-nsc-windows-x64-release-master CI: Promote NSC channel to 66da590

devshgraphicsprogramming mentioned this pull request Mar 23, 2026

Address path tracer merge leftover comments #991

Merged

keptsecret added 5 commits March 24, 2026 10:03

merge master, fix conflicts

95a3862

pass ray to nee generate instead, store depth in ray

946c9c7

nee deferredPdf is deferredWeight

4f4421d

all bxdfs have a cache type from generate to quotient_weight, microfa…

a70f420

…cet checks cache with concept

eval should have its own cache type in concepts

dab4a18

devshgraphicsprogramming commented Mar 24, 2026

View reviewed changes

keptsecret added 5 commits March 25, 2026 10:49

minor typo bug fixes

fb878a6

eval returns a value_and_weight type, fixed some bugs in bxdf concept

4171059

made value_and_pdf compose from value_and_weight

0e16279

reuse quant query and other stuff from forwardPdf in quotient and eva…

345865f

…l methods

reuse reflectance from forwardpdf in eval

a98cd45

devshgraphicsprogramming commented Mar 25, 2026

View reviewed changes

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl Show resolved Hide resolved

devshgraphicsprogramming commented Mar 25, 2026

View reviewed changes

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl Show resolved Hide resolved

devshgraphicsprogramming commented Mar 25, 2026

View reviewed changes

include/nbl/builtin/hlsl/bxdf/base/cook_torrance_base.hlsl Show resolved Hide resolved

devshgraphicsprogramming mentioned this pull request Mar 25, 2026

Env map importance sampling #969

Open

keptsecret added 5 commits March 26, 2026 10:41

renamed quotient_pdf type to quotient_weight type

d0ed7d3

minor fixes to cook torrance, removed redundant functions

110547a

only microfacet bxdfs need eval cache type, allow for overload with e…

ef9156e

…xisting cache type too

latest example

05124c2

latest example

258836a

devshgraphicsprogramming commented Mar 26, 2026

View reviewed changes

devshgraphicsprogramming and others added 4 commits March 26, 2026 11:30

fixes in HLSL, SPIR-V intrinsics and Mitsuba parsing merged

01ee867

update submodule pointer after merge

e11b118

latest example

efa17ca

minor changes to concepts, microfacet bxdfs have eval that autogenera…

bea4780

…tes cache

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Resampled Importance Sampling bxdfs#1027

Resampled Importance Sampling bxdfs#1027
devshgraphicsprogramming wants to merge 64 commits intosampler-conceptsfrom
ris_bxdfs

devshgraphicsprogramming commented Mar 20, 2026

Uh oh!

devshgraphicsprogramming Mar 24, 2026

Uh oh!

devshgraphicsprogramming Mar 24, 2026

Uh oh!

devshgraphicsprogramming Mar 24, 2026

Uh oh!

Uh oh!

Uh oh!

devshgraphicsprogramming Mar 25, 2026

Uh oh!

keptsecret Mar 26, 2026

Uh oh!

devshgraphicsprogramming Mar 26, 2026

Uh oh!

Uh oh!

devshgraphicsprogramming Mar 26, 2026

Uh oh!

devshgraphicsprogramming Mar 26, 2026

Uh oh!

devshgraphicsprogramming Mar 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

devshgraphicsprogramming commented Mar 20, 2026

Description

Testing

TODO list:

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants