Skip to content

Hlsl path tracer #188

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 325 commits into
base: bxdf_unit_tests
Choose a base branch
from
Open

Hlsl path tracer #188

wants to merge 325 commits into from

Conversation

devshgraphicsprogramming
Copy link
Member

No description provided.

kevyuu and others added 30 commits March 12, 2025 22:56
Fix merge bug.
6517442
Fix demo to use the new traceRayIndirect
6465312
@keptsecret
fixed nan and accumulation going black problem
6abb635
@keptsecret
fixed triangle light, rectangle needs checking
1eee3ca
@keptsecret
simplified material data
011fbfb
Small fixes on ray trace pipeline demo.
4d1dca4
Optimize ray tracing demo occlusion tracing
cc84091
@keptsecret
made scene a static global var
63b64e3
@keptsecret
fixed most of rectangle light issues, still red pixels
7bd69e9
Use unused shader instead of stub shader for occlusion ray miss shader.
38d8285
@keptsecret
fix for nan samples
ab0aa12
@keptsecret
revert to intial scene settings
96c7497
@keptsecret
Merge branch 'master' into hlsl_path_tracer
b5194ef
Use nbl::hlsl::_static_cast for converting MaterialPacked to Material…
f96dfcc 
… and vice versa
Fix create shader binding table to use the new span api
88b3275
Some optimization on ray tracing demo
cca8f72
@keptsecret
better hlsl dispatch
b483aa6
@keptsecret
refactor NEE to use templated light types and sampling
773733d
@keptsecret
use 1D workgroup dispatch
b889b60
@keptsecret
removed obsolete commented sections
79ee9da
@keptsecret
some minor corrections
ca8f2ec
Return materialId instead of materialPacked from rchit
c904134
Add ray tracing no null flags
6b3ae54
Add setRayTracingStackSize in the demo
b2abf00
@keptsecret
changed workgroup size to 512
e95f09d
@keptsecret
workgroup size 512 for sure this time
56994a9
@keptsecret
use morton and virtual indexing
3cdfb4b
@keptsecret
reverted virtual index, fix hlsl colors
5f93cec
@keptsecret
fixed some bugs for cpp compat
78de4f5
@keptsecret
initial benchmark example copy
8090a2d
devsh and others added 30 commits June 19, 2025 02:19
note down embedding TODOs for arek
a347562
remove unused shaders
cf4e279
get the example to find shaders and create pipelines, but nothing on …
08c28ef 
…screen
scene was empty, no wonder nothing drew
17b0579
index type was not being set
62f1a26
Initial commit for example 71 to use ICPUPolygonGeometry
fcae0b4
@AnastaZIuk
pull Matt and resolve conflicts
312ca61
@AnastaZIuk
docs docs docs, adjust to comments and test on 01 after updates
e419580
matrix math in linalg was fine, I made a typo
cfd609c
fix a caption bug
6242357
Merge remote-tracking branch 'remotes/origin/mlBuildUpdates' into mes…
19df6e1 
…h_loaders
leave pointers on how to port example 62_CAD, also EXCLUDE_FROM_ALL…
19167c5 
… the examples which are WIP on this branch
kill old meshloaders example that didn't work since forever
a0622f3
yes NBL_EMBED_BUILTIN_RESOURCES should control both Nable and examp…
f494d85 
…les_tests w.r.t. the question of `MonoAssetManagerAndBuiltinResourceApplication.hpp`

There really isn't a usecase for:
- embedding Nabla resources but not embedding example (there's no harm in Nabla resources not being embedded)
- even more strangely, not embedding Nabla resources, but embedding example.
Adjust all examples after splitting `MonoAssetManagerAndBuiltinResour…
ce268fb 
…ceApplication` in two
correct namespace ambiguities affecting example 23 and 29
20a5438
aaah the BuiltinResourcesApplication.hpp header needs some special …
21a88ff 
…treatment to NOT include `this_example/CArchive`
make the BuiltinResourcesApplication work for examples without buil…
1b3c19c 
…tins too
make CSwapchainFramebuffersAndDepth support runtime depth buffer resi…
7ec6846 
…gnation
remake example 61, not thoroughly tested, some TODOs remain for @Anas…
64e7b26 
…taZIuk
fix bugs in ex 61:
2872604 
- correct aspect masks on image views
- wrong renderpass given to imgui
- handle virtual window getting minimized
- imguizmo not updating
- imgui not drawing offscreen image
Push Mesh Loader example
20cc57e
decouple CSimpleDebugRenderer from CGeometryCreatorScene
86cc7dd
more improvements to make the Simple Debug Renderer more runtime frie…
e790c84 
…ndly
prep the conversion
eaa1320
just some todo markup
5929be1
@keptsecret
ready changes for mesh_loaders merge, requires examples.hpp from mesh…
52c1aa5 
…_loaders
@keptsecret
merge mesh_loaders, fix conflict
d596f75
@keptsecret
cpp fixes so it compiles at least
c43c93b
@keptsecret
a bazillion fixes since last time bxdf usages changed
8b31859
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@devshgraphicsprogramming @AnastaZIuk @keptsecret