Bugzilla – Bug 9287
Light sensor test in FW ugly
Last modified: 2008-09-15 09:51:34 UTC
right now it says: Ambient light (lux): 1 @ nnn/10+2=2 +0=2 Felix notes in bug 9104 that it's not calibrated, so it's really not lux. In addition, it's ridiculously user-un-friendly. I'm the QA manager and I don't know what it means! I don't care if you give me one number or two or three, but give me something that makes sense. A=nnn B=mmm T=xxx would be a wonderful improvement instead of something that looks like someone forgot to take out their printf debug statements.
I fully agree with you, it looks like a debug statement and you know why? It is kind of one. ;) 1 @ nnn/10+2=2 +0=2 The first letter is the type of ambient light sensor: '1' => old, 'b' => new (So I hope you got that line from an old Boom unit.) The '@' with cursor is there to see how the gray scale looks like. 'nnn' => is the raw value from the lightsensor The next two numbers '10' and '2' are values that can be changed from SC and used so get to the final value used in fw to control the display brightness. You can play with this numbers using an old remote: '1', '4', '7' increase, default and decrease the first number. '2', '5', '8' increase, default and decrease the second number. The last part is controlling the backlights of the buttons: '+0' is the offset from what the display is currently at and the last value is used in fw to control the button brightness. Again '3', '6' and '9' on the remote control the button backlight offset. As soon as we have found a good formula that works for all Booms, we can replace this with something more user friendly.