secondlife / viewer

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

increase maximum Exposure value in preferences to 4.0 #2916

Open brad-linden opened 1 month ago

brad-linden commented 1 month ago

Description

increase max exposure value in preferences to 4.0 And default exposure should now be 1.5 is 1.0

brad-linden commented 1 month ago

implemented in PR #2915

igorlproductengine commented 1 month 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.

1. Inconsistency in Maximum Values: In the Preferences > Graphics tab, the maximum value for the Brightness (exposure) parameter is set to 4.0. However, when opening the Advanced settings floater, the maximum value for the Exposure parameter is 1.5 instead of 4.0, creating a mismatch between these two settings. Image

2. Issue with Resetting Settings: Pressing the "Reset to recommended settings" button does not reset the Exposure value to the default of 1.5, which is the expected behavior. This forces users to manually adjust the settings.

3. Visual Perception and Other Observations: The standard environment looks better with an exposure setting of 1.2. The Exposure setting does not affect the "World > Environment > Midday (Legacy)" option.

AtlasLinden commented 1 month ago

1.5 determined to be too bright. Further discussions to be made on if this should be made 1.2 or a different value. For now please revert back to 1.0.

igorlproductengine commented 1 month ago

Failed 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.

1. Inconsistency in Maximum Values: In the Preferences > Graphics tab, the maximum value for the Brightness (exposure) parameter is set to 4.0. However, when opening the Advanced settings floater, the maximum value for the Exposure parameter is 1.5 instead of 4.0, creating a mismatch between these two settings.

2. Default Value Clarification: The current default value for exposure has now been set to 1.0.

dmytrohproductengine commented 1 week ago

Passed QA.

Verified on the Second Life Release 7.1.11.11982653819 (64bit) on Win10/OSX in the scope of IQA#395.