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

Add new PBR Neutral tone mapper #7597

Merged
merged 1 commit into from Feb 23, 2024
Merged

Add new PBR Neutral tone mapper #7597

merged 1 commit into from Feb 23, 2024

Conversation

romainguy
Copy link
Collaborator

@romainguy romainguy commented Feb 23, 2024

This tone mapper was designed to preserve the color appearance of materials. It provides good saturation and contrast while controlling the highlights. This new tone mapper solves (most of) the color skews found in ACES and provides a good path to white similar to AgX but with better preservation of contrast and saturation.

PBR Neutral is a good choice to showcase products or any type of scene where the appearance of the colors is important (it was designed for commerce applications).

MacBeth color palette in a white furnace

This test compares lit and unlit materials, and the colors should match as closely as possible.

ACES
Screenshot 2024-02-23 at 10 47 30 AM

AgX
Screenshot 2024-02-23 at 10 47 37 AM

PBR Neutral
Screenshot 2024-02-23 at 10 47 39 AM

Color sweep

ACES
Screenshot 2024-02-23 at 10 49 25 AM

AgX
Screenshot 2024-02-23 at 10 49 27 AM

PBR Neutral
Screenshot 2024-02-23 at 10 49 30 AM

Bistro at night

ACES
aces

AgX
agx

PBR Neutral
pbrneutral

This tone mapper was designed to preserve the color apperance
of materials. It provides good saturation and contrast while
controlling the highlights.
@romainguy romainguy merged commit 3612010 into main Feb 23, 2024
11 checks passed
@romainguy romainguy deleted the rg/neutral branch February 23, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants