Skip to content

Conversation

@aevyrie
Copy link
Member

@aevyrie aevyrie commented Jan 14, 2026

Objective

  • Unbreak fresnel.
  • Fix a small math error.

Solution

  • Don't be stupid.

Testing

main: fresnel missing entirely

image

Roughness dependent fresnel: still too bright

image

Note how this violates the furnace test (reflection is brighter (has more energy) than what it is reflecting):

image

Fixed existing, boring fresnel: juuuuust right

image

and this one does not

image

Bistro

More fresnel than the 0.18 release post, but it seems like a correct amount, considering it lacks any specular occlusion.

Release post
image

This PR (light intensities different, doesn't affect fresnel)
image

This PR + SSR - fresnel is further occluded where it should be
image

As soon as you add specular occlusion with SSR, the fresnel starts to make sense again, occluded objects no longer reflect the envmap in their fresnel reflection. Note the awnings here look bright in shadow because they are catching the envmap, until you turn on SSR which tones this down with specular occlusion.

image-3 bevyssr

Note that the fresnel is actually helpful here - without it we wouldn't get this soft reflection of the scooter against the sky (env) on the pavers:

image image

@aevyrie aevyrie changed the title I want to get of Mr. Fresnel's wild ride I want to get off Mr. Fresnel's wild ride Jan 14, 2026
@alice-i-cecile alice-i-cecile added this to the 0.18.1 milestone Jan 14, 2026
@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 14, 2026
@atlv24 atlv24 added the M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered label Jan 14, 2026
@alice-i-cecile
Copy link
Member

alice-i-cecile commented Jan 14, 2026

Do we want to try the same comparison but with the Blender sphere reference scene? Or is that not suitable for examining fresnel specifically?

@aevyrie
Copy link
Member Author

aevyrie commented Jan 14, 2026

The main reference we have for this is the gray furnace test, which we seem to be passing still, without completely deleting fresnel.

@alice-i-cecile alice-i-cecile requested a review from atlv24 January 14, 2026 18:34
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jan 14, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jan 14, 2026
Merged via the queue into bevyengine:main with commit e42e183 Jan 14, 2026
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Bug An unexpected or incorrect behavior M-Deliberate-Rendering-Change An intentional change to how tests and examples are rendered S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants