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

PBR materials: Fix refraction color calculation #16357

Closed
wants to merge 1 commit into from

Conversation

Popov72
Copy link
Contributor

@Popov72 Popov72 commented Mar 18, 2025

We incorrectly scaled the refraction color with the environment map intensity.

As a result, I reverted #10537, which was the wrong fix.

Note that this is a breaking change, but with limited consequences because it can only occur when scene.environmentIntensity != 1 and when using refraction in a PBR material.

@bjsplat
Copy link
Collaborator

bjsplat commented Mar 18, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@deltakosh deltakosh enabled auto-merge (squash) March 18, 2025 23:50
@bjsplat
Copy link
Collaborator

bjsplat commented Mar 19, 2025

Copy link
Member

@sebavan sebavan left a comment

Choose a reason for hiding this comment

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

Is vLightingIntensity.z already factored in ?

@Popov72 Popov72 marked this pull request as draft March 19, 2025 00:50
auto-merge was automatically disabled March 19, 2025 00:50

Pull request was converted to draft

@bjsplat
Copy link
Collaborator

bjsplat commented Mar 19, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Mar 19, 2025

@Popov72
Copy link
Contributor Author

Popov72 commented Mar 19, 2025

Discussed with Seb, it's not the way to fix it. I will probably do another fix.

@Popov72 Popov72 closed this Mar 19, 2025
@Popov72 Popov72 mentioned this pull request Mar 19, 2025
sebavan pushed a commit that referenced this pull request Mar 20, 2025
In replacement of #16357.

The inspector's “IBL Intensity” slider now correctly changes the
intensity of the IBL and not the intensity of the indirect lighting.
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.

4 participants