Page MenuHome

Equi-Angular Cubemap face rendering
ClosedPublic

Authored by Damian Trebilco (Damian_Trebilco) on Dec 9 2021, 1:46 PM.

Details

Summary

Add the ability to render a equiangular cube map single face as described in
https://blog.google/products/google-ar-vr/bringing-pixels-front-and-center-vr-video/

This was added as I found rendering directly in panorama space caused artefacts in denoising as the denoising occurs in 2D space.
Direct 6 cube faces rendering works with custom denoising, but I wanted slightly more quality.

Diff Detail

Repository
rC Cycles

Event Timeline

Damian Trebilco (Damian_Trebilco) requested review of this revision.Dec 9 2021, 1:46 PM
Damian Trebilco (Damian_Trebilco) created this revision.
Brecht Van Lommel (brecht) changed the repository for this revision from rC Cycles to rB Blender.Dec 9 2021, 8:16 PM
Brecht Van Lommel (brecht) requested changes to this revision.Dec 9 2021, 8:23 PM

This seems useful. It's pretty verbose, but we could call this Equiangular Cubemap Face to make it really clear what this is.

Would it make sense to add a regular "Cubemap Face" as well? Or is that just the same as one of the existing options?

direction_to_equiangle should be the inverse of equiangle_to_direction. It is used for Texture Coordinate > Window in the shader nodes. The result should be 0..1 across the image, as it is for other cameras.

This revision now requires changes to proceed.Dec 9 2021, 8:23 PM
Damian Trebilco (Damian_Trebilco) edited the summary of this revision. (Show Details)

Renamed the enum to be more verbose - EQUIANGLE_CUBEMAP_FACE
Verified that the math in direction_to_equiangle reverses equiangle_to_direction

A separate basic cube map face setting should not be needed as setting a basic projection camera to 90 degrees does this.

I also noticed other people are requesting this projection.

(I know this is for eevee however).
https://blender.community/c/rightclickselect/3Sfbbc/?sorting=hot

@Brecht Van Lommel (brecht) Would you be able to tell me what I could do to get this change merged?

Thanks for the updates, I'll commit this with some naming / description tweaks.

This revision is now accepted and ready to land.Oct 28 2022, 4:20 PM