Skip to content

feature: WIP occlusion culling#137

Merged
schell merged 10 commits intomainfrom
feature/occlusion-culling
Nov 15, 2024
Merged

feature: WIP occlusion culling#137
schell merged 10 commits intomainfrom
feature/occlusion-culling

Conversation

@schell
Copy link
Owner

@schell schell commented Oct 4, 2024

This is an extension from #130 to add occlusion culling to the compute culling machinery.

added MipMapGenerator, depth pyramid shaders

slab buffers are created with size 1 to avoid wgpu validation bug

fixed a bug where re-staging a texture often copied to the wrong layer

debbugging led to better sphere projection for culling

update sample count so we don't invalidate every frame

debug overlay of bounding spheres

use box instead of circle for bounds in debug overlay

don't use compute culling by default
@schell schell force-pushed the feature/occlusion-culling branch from 43d6bab to a2215c4 Compare November 11, 2024 07:26
@schell schell marked this pull request as ready for review November 15, 2024 22:45
@schell schell merged commit 61eecac into main Nov 15, 2024
@schell schell deleted the feature/occlusion-culling branch November 15, 2024 23:05
schell added a commit that referenced this pull request Dec 27, 2024
* use WeakHybrid in InternalRenderlet

* Occlusion culling, part one

added MipMapGenerator, depth pyramid shaders

slab buffers are created with size 1 to avoid wgpu validation bug

fixed a bug where re-staging a texture often copied to the wrong layer

debbugging led to better sphere projection for culling

update sample count so we don't invalidate every frame

debug overlay of bounding spheres

use box instead of circle for bounds in debug overlay

don't use compute culling by default

* regenerate shaders, fix merge errors, treat *.spv and *.wgsl as binary

* use cargo-gpu on branch

* clean rust-gpu checkouts (temporary)

* also clean the rust-gpu cache dir

* allow the cache to persist

* don't use bool as u32 in shaders

* change rust-gpu cache step

* update cargo gpu
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.

1 participant