Skip to content

Conversation

DarioSamo
Copy link
Contributor

This fixes an issue where the size of the downsampled radiance map was not updated properly for reflection probes when changing between scenes, leading to much lower quality visuals if the reflection probe it happens to reuse was created first with the real time mode instead. Viceversa, the real time probe would end up much higher quality than intended if an scene was opened with a probe that used the Update Once method instead.

Both screenshots here were taken with the same scene, the main difference being that a different scene was opened first that had a reflection probe configured in Update Once mode and the other scene was in Update Always mode.

image image

@DarioSamo DarioSamo requested a review from a team as a code owner September 8, 2025 18:57
@clayjohn clayjohn added this to the 4.6 milestone Sep 8, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, it works as expected. Code looks good to me. (I've tested this with texture array reflections disabled as well, just in case.)

Testing project: test_reflection_probe_once_always.zip

I was able to reproduce the original issue on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants