New issue
Advanced search Search tips

Issue 723249 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Image Capture: |exposureCompensation| has too many decimals

Project Member Reported by mcasas@chromium.org, May 17 2017

Issue description

getCapabilities().exposureCompensation step is 0.1666666616337204 on a Nexus 6. It may be better to simply round it to 0.16.
 

Comment 1 by mcasas@chromium.org, May 17 2017

Owner: mcasas@chromium.org
Status: Started (was: Available)
Why not round to 0.17? I'm not convinced of the necessity of this change.
I just discussed this offline with mcasas@ and we agreed we need fbeaufort@'s input on the developer impact of this change. My argument is that by clamping this value to 2 decimal places we are introducing false precision that will make it harder for developers to display human-readable values to the user. The reason for this is that when a step of 0.1666666616337204 is multiplied by a slider position, say, 6, we get 0.9999999698023224. Script can choose to round this to 2 decimal places and show the value the user expects, 1.00. If we use a step of 0.17 then that same logic will display 1.02.

Comment 4 by mcasas@chromium.org, May 18 2017

Cc: reillyg@chromium.org
Status: WontFix (was: Started)
Marking as Won't Fix after the convincing argument of reillyg@.

Sign in to add a comment