Skip to content
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

Add DDGI for global illumination #243

Merged
merged 84 commits into from
Jan 27, 2025
Merged

Add DDGI for global illumination #243

merged 84 commits into from
Jan 27, 2025

Conversation

Duttenheim
Copy link
Contributor

Probe based global illumination using DDGI. Supports:

  • Adjustable probe count
  • Adjustable rays per probe
  • Update budget, decide how many % of probes should be updated per frame
  • Classification to turn off updates for unnecessary probes
  • Relocation to make probes adjust to their environment
  • Only compatible with raytracing
  • Debug with the g_debug_ddgi CVAR

Comes with a bunch of fixes to the raytracing, mainly:

  • Default hit shaders provide radiance - light calculation is done on hit
  • Material updates are seen in the raytracing system when changing them using the editor

@Duttenheim Duttenheim self-assigned this Jan 17, 2025
@Duttenheim
Continue work on DDGI
f8e0f0c
@Duttenheim
Move sample counts to globals in shader to allow for setting up the p…
8c6926b 
…robe buffers CPU side
@Duttenheim
Continued DDGI work
00b8bb3
@Duttenheim
DDGI volume deletion
9d6b669 
Selection tool scales with size of item, even if not a model
Fixed error in component inspection that would recreate an item indefinitely when a uint value was defined
@Duttenheim
Split ddgi debug to own shader
4ddf689
@Duttenheim
Continued DDGI work. Initial probe update close to finished.
d7d36a3
@Duttenheim
Fix ratio calculation and add range adjustment for texture preview
2d6a2cb
@Duttenheim
Start working on step 2 of DDGI, finalizing the light contributions
f20fabf
@Duttenheim
Continued work on finalization shader
9afc428
@Duttenheim
Continue work on probe finalize
2a22364
@Duttenheim
More work on probe finalization
ee1fb64
@Duttenheim
More DDGI stuff, feeding the blending shader and syncing resources.
d61cf04
@Duttenheim
Minor fixes
358f89f
@Duttenheim
Add DDGI to clustering system
5310810
@Duttenheim
Add color masking to Imgui shader
d91e217
@Duttenheim
Add more DDGI settings
0281560
@Duttenheim
Add switch to enable/disable terrain shadows
3e432f0
@Duttenheim
Continued DDGI work and added script to convert old surface definitio…
9025d55 
…ns to new.
@Duttenheim
Add mode for converting an entire folder to the surface updater
9e8807c
@Duttenheim
Make DDGI buffers full precision when working
0a939ef
@Duttenheim
Raytracing fixes
c3ae01a
@Duttenheim
Fix resource server not picking the correct loader
1c57b62
@Duttenheim
Fix bugs in material parsing
0ff3ec3
@Duttenheim
Make area lights cull in view space in shader, but passed in world space
c26c0b9
@Duttenheim
Add texture channel masking to resource browser
b7ef0b6
@Duttenheim
Fixed typo
f5b83ef
@Duttenheim
Bind ray tracing pipeline before binding resources to ensure the layo…
2fbc59a 
…ut change doesn't invalidate the bindings
@Duttenheim
Ensure we can't have less than the lowest required amount of rays per…
c89fb21 
… probe
@Duttenheim
More fixes for lowest amount of allowed rays per probe
366671d
@Duttenheim
Fix build
7e8bb9a
@Duttenheim
More build fixes
ad3fd39
@Duttenheim
Fix test apps with new API for setting up globallight (w/o backlight …
f575198 
…garbage)
@Duttenheim
Add base level and fix include of old terrain settings.
49293f7 
Fix DDGI compile issue
@Duttenheim
Working on fixing a raytracing bug and fixing some stupid decisions w…
d34ca96 
…ith mesh exporting
@Duttenheim
Fix physics with the new nvx format
a3220a2
@Duttenheim
Use uint64 for GPU offsets and sizes
6727462 
Also limit uniform buffer binding to size min(MaxUniformSize, buffer.size)
@Duttenheim
Fix graphics manager not triggering DDGI if raytracing is off
9dc4682
@Duttenheim
Some raytracing fixes
6364200
@Duttenheim
Revert max vertex count
f4b90a9
@Duttenheim
Working on fixing what appears to be incorrect material assignment wh…
9656ecb 
…en raytracing.
@Duttenheim
Fix raytracing issues where a model with multiple primitive groups wo…
7a2d7d3 
…uldn't render correctly.
@Duttenheim
Fix mesh unload double delete
582d531
@Duttenheim
Improve DDGI debugging and safe guard against negative energy
2cf9354
@Duttenheim
Material template generator spits out 16 bit aligned material types w…
0b4d067 
…hen used as pointers.
@Duttenheim
Remove StackAlloc in favor of ArrayAllocStack, which is a thread loca…
69195df 
…l linear buffer.
@Duttenheim
Fix some export issues
c91cca5
@Duttenheim
Material asset editor allows for copy-paste of textures using right c…
8272014 
…lick (copy) and middle click (paste)
@Duttenheim
Correctly scale selection boxes based on object scale
c833db5
@Duttenheim
Minor fixes in material loader, mainly renaming the global state to m…
43684a0 
…aterialLoaderState to not confuse debuggers (didn't work)
@Duttenheim
Disable point sampling for mip mapped normal maps. Technically normal…
489a838 
… maps "can't" be mipped but whatever...
@Duttenheim
Add DDGI update frequency to project settings
b0c6916
@Duttenheim
New syswork export with the changes done to the content
f3b31e3
@Duttenheim
Fix bug with light grid where lights didn't have enough cells to be c…
dfcb2fb 
…ulled into. Miracle it ever worked.

I think this could be solved in a better way, but it will work for now.
@Duttenheim
Turn on raytracing by default on models. Only does something if raytr…
3951a20 
…acing is enabled anyways.
@Duttenheim Duttenheim merged commit d8e45fb into master Jan 27, 2025
4 checks passed
@Duttenheim Duttenheim deleted the ddgi branch January 27, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Duttenheim Duttenheim

Labels
category: experiment Experiments category: graphics Graphics
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Duttenheim