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.

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

Voxelgi with clipmaps. #2990

Closed
wants to merge 3 commits into from
Closed

Voxelgi with clipmaps. #2990

wants to merge 3 commits into from

Conversation

e2002e
Copy link
Contributor

@e2002e e2002e commented Jan 19, 2024

This pull request implements VoxelGI with clipmaps (working with AO too).
The image used to store the voxels is scaled on the y axis by the number of clipmaps, and each level of detail is stored at a different offset, so that we just have to compute the offset in the conetracer according to the distance to the camera, so the clipmap_center is a new parameter for tracing functions.

I removed the snapping mechanism because it was making seams and it already occurs in the conetracer.
We don't compute voxpos anymore, only the clipmap_center (eye + eyeLook) is passed to the conetracing function.

The dimension option, revoxelize, and 'around camera' where removed, since clipmapping depends on the distance to the camera. The new way of computing dimension is 'voxelSize * voxelgiResolution'. You'll see the new option size in the ray panel ; it determines the size of each voxels thus the precision of the voxelization.

Second bounce is working but it's not compatible with the temporal filter, I don't know why it's doing some splashes and smears. This is the only bug.

There is the forum's thread with some renders:
https://forums.armory3d.org/t/voxel-gi-with-clipmaps/5242/5

@e2002e
Copy link
Contributor Author

e2002e commented Jan 20, 2024

I'm closing (will rewrite for compute shaders)

@e2002e e2002e closed this Jan 20, 2024
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