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

Skip expensive ambient light computation on fully metallic materials #103854

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tygyh
Copy link
Contributor

@tygyh tygyh commented Mar 9, 2025

Replaces #58635

Description

This improves performance significantly in scenes that have fully metallic materials (metallic = 1.0).

Fully metallic and smooth materials (roughness = 1.0, metallic = 1.0) bring further performance improvements by also disabling shadow map sampling.

@tygyh tygyh requested a review from a team as a code owner March 9, 2025 09:42
@tygyh tygyh force-pushed the metallic-smooth-skip-ambient-computation branch 5 times, most recently from d6178fb to cca9e52 Compare March 9, 2025 13:01
This improves performance significantly in scenes that have
fully metallic materials (metallic = 1.0).

Fully metallic *and* smooth materials (roughness = 1.0, metallic = 1.0)
bring further performance improvements by also disabling shadow map
sampling.
@tygyh tygyh force-pushed the metallic-smooth-skip-ambient-computation branch from cca9e52 to 7c4f755 Compare March 9, 2025 13:10
@Calinou Calinou added this to the 4.x milestone Mar 9, 2025
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.

2 participants