Skip to content

HDR Tonemap Modified Reinhard equation possibly wrong #12175

Open
@jjspace

Description

@jjspace

I think the current equation for our Modified Reinhard tonemap is slightly wrong

color = (color * (1.0 + color / white)) / (1.0 + color);

The original source link seems to no longer exist but judging by these other articles I found the color / white should be color / (white * white)

This hasn't been touched since the initial HDR PR (#7017) so I don't know the context here that might dictate this as intentional.

This came up during #12160

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions