[Bug 1618391] Re: Brightness is too strong

[Launchpad Bug Tracker]

This bug was fixed in the package repowerd -
2016.10+16.10.20161007-0ubuntu1

---
repowerd (2016.10+16.10.20161007-0ubuntu1) yakkety; urgency=medium

  * New 2016.10 release
- Fix typo in brightness config value (LP: #1618391)
- Update repowerd-cli help text
- Handle SIGTERM gracefully in repowerd-cli
- Wait for the repowerd DBus API to be accessible before marking
  the upstart job as started (LP: #1623853)
  * 2016.10 release

 -- Alexandros Frantzis   Fri, 07 Oct
2016 11:50:44 +

** Changed in: repowerd (Ubuntu)
   Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Alexandros Frantzis]

> Suggested steps to improve the situation:
>
> * Fix typo in repowerd (assigned to me)

The fix has been released in repowerd 2016.10

> * Change device specific config files to have lower minimum values for
both (manual) brightness setting and the autobrightness curve.

This is not done yet. Not sure which the best team to handle this.

** Changed in: canonical-devices-system-image
   Status: Fix Committed => Confirmed

** Changed in: repowerd (Ubuntu)
   Status: Confirmed => Fix Committed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Jean-Baptiste Lallement]

** Changed in: canonical-devices-system-image
   Status: Confirmed => Fix Committed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Launchpad Bug Tracker]

Status changed to 'Confirmed' because the bug affects multiple users.

** Changed in: repowerd (Ubuntu)
   Status: New => Confirmed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Alexandros Frantzis]

> In experimenting with this I noticed a typo in the source when getting this 
> parameter, so the min
> will always be 10 on every system in spite of the config.

Thanks Pat, this is indeed a typo in repowerd.

That being said, two things to note in relation to the "brightness is
too strong":

1. The screenBrightnessSettingMininum config has no effect on
autobrightness; it only affects the minimum value manually settable by
the user. Autobrightness is governed by the curve specified in
"config_autoBrightnessLcdBacklightValues" in the device specific config
file (/usr/share/powerd/device_configs/config-default.xml)

2. By pure chance, this bug has no effect on E4.5 and allows an even
darker setting on MX4 (not sure about PRO5) for the manual case, since
it defaults to 10 for min brightness, whereas the actual config values
for the aforementioned devices are 10 and 15 respectively.

Suggested steps to improve the situation:

* Fix typo in repowerd (assigned to me)
* Change device specific config files to have lower minimum values for both 
(manual) brightness setting and the autobrightness curve.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Pat McGowan]

The devices are all using a default value (in this case 10). We could
check with the vendors to see if they have overriden the defaults for
other OSes.

In experimenting with this I noticed a typo in the source when getting this 
parameter, so the min will always be 10 on every system in spite of the config. 
in src/adapters/brightness_params.cpp 
auto min_str = device_config.get("screenBrightnessSettingMininum", "10");
should be ...Minimum

** Changed in: canonical-devices-system-image
   Status: Incomplete => Confirmed

** Changed in: repowerd (Ubuntu)
 Assignee: (unassigned) => Alexandros Frantzis (afrantzis)

** Changed in: canonical-devices-system-image
   Importance: Undecided => Medium

** Changed in: canonical-devices-system-image
Milestone: None => backlog

** Changed in: canonical-devices-system-image
 Assignee: (unassigned) => Pat McGowan (pat-mcgowan)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1618391] Re: Brightness is too strong

[Pat McGowan]

** Also affects: repowerd (Ubuntu)
   Importance: Undecided
   Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1618391

Title:
  Brightness is too strong

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1618391/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs