secondlife / viewer

🖥️ Second Life's official client
GNU Lesser General Public License v2.1
211 stars 53 forks source link

Reset tone mapper default back to ACES #2913

Open AtlasLinden opened 3 weeks ago

AtlasLinden commented 3 weeks ago

Environment

ExtraFPS

Description

Reset default tone mapper to ACES instead of Khronos Neutral. This also includes setting Tone Map Mix to 70% as it was previously.

Reproduction steps

Reset graphics to default settings Observe Tone Mapper settings in Advanced Graphics Preferences

igorlproductengine commented 3 weeks ago

Failed QA

Verified on the Second Life Release 7.1.11.11469579464 (64bit) on Win10/OSX in the scope of https://github.com/secondlife/iqa/issues/374.

Currently, the "Reset to recommended settings" button does not affect the Tone Mapper settings. After manually changing both the Tone Mapper and Tone Map Mix values, applying the reset had no impact, and the settings remained unchanged.

canny[bot] commented 3 weeks ago

This issue has been linked to a Canny post: [ExtraFPS] Release Candidate defaults to Khronos for tone mapping breaking existing EEP and Content in PBR/Non PBR :tada:

igorlproductengine commented 2 weeks ago

Passed QA.

Verified on the Second Life Release 7.1.11.11565212741 (64bit) on Win10/OSX in the scope of https://github.com/secondlife/iqa/issues/375.

canny[bot] commented 1 week ago

This issue has been linked to a Canny post: [ExtraFPS] Khronos neutral tone mapping has odd high end :tada: