Bug 1850 - Feature request: Brightness timer
: Feature request: Brightness timer
Status: RESOLVED DUPLICATE of bug 8237
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: unspecified
: All All
: P2 enhancement with 1 vote (vote)
: 7.x
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-07-22 00:15 UTC by Bart
Modified: 2009-07-31 10:13 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Eric Koldinger's original 7.0 port of AutoDisplay (12.29 KB, text/plain)
2008-05-26 23:34 UTC, Daryle Tilroe
Details
Some slight mods to Eric's 7.0 port of AutoDisplay (12.29 KB, text/plain)
2008-05-26 23:34 UTC, Daryle Tilroe
Details
Daryle's back revision to polling of Eric's 7.0 port of AutoDisplay (11.72 KB, text/plain)
2008-05-26 23:35 UTC, Daryle Tilroe
Details
Daryle's revision to modular arithmetic comparisions/calculations of Eric's 7.0 port of AutoDisplay (12.39 KB, text/plain)
2008-05-26 23:36 UTC, Daryle Tilroe
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Bart 2005-07-22 00:15:11 UTC
Would it be possible to add an option to control the brightness level of the 
display, depending on whether it is playing, time of day, time of year and 
user preferences?
eg code which, knowing your time zone and date, dims the display at night and 
brightens it 30 minutes after sunrise?
Comment 1 KDF 2005-07-22 00:29:45 UTC
sounds like a perfect plan for a plugin. it oculd even include a lookup for the
proper sunrise time.  
Comment 2 Daryle Tilroe 2005-10-30 00:00:01 UTC
I was going to request this enhancement myself.  One could easily go nuts and
have sophisticated timers on all the states; however for me the main reason for
this is to have the SB display go blank at night on my bedroom unit when turned
off.  Perhaps a single timer on the 'off' state with a start and stop time. 
This could be for full blank or a selectable -x adjustment.  In that latter case
you could also have radio buttons on all three states to select the adjustment.
Comment 3 KDF 2005-10-30 00:32:34 UTC
The AutoDisplay plugin should handle the simple case (or perhaps you can ask
felix for slight changes):
http://www (dot) gwendesign (dot) com/slimserver/development.htm#autodisplay
Comment 4 Daryle Tilroe 2005-10-30 01:01:28 UTC
Thanks for the pointer kdf!  I should have looked at all the plugins myself but
it's a good thing to have it referenced in this enhancment request as well.  It
would probably be a nice feature to eventually roll into the main software
however the plugin will likely do the job for now.
Comment 5 Daryle Tilroe 2005-11-05 10:37:11 UTC
I just thought I would document here that I fixed a tiny bug and further tweaked
autodisplay.pm a bit.  Felix has kindly posted it as an alternate version (0.4a)
for now:

http://www.gwendesign.ch/slimserver/dev_software.htm#autodisplay

I just made it so if the power was on it would not blank (because if it is on it
is presumed someone is using it) and also so it would transition/change state
any time in the two time windows (one on and one off in 24hrs), not just at the
rollover times.  Argueably this would be a nice feature to roll into the release
since, at least in a pitch dark bedroom, the display can be quite bright, even
at its dimmest setting and it allows you to have the off brightness setting
slightly brighter for better reading of the clock or rss feeds in daylight.
Comment 6 Daryle Tilroe 2008-05-26 23:31:47 UTC
As noted here:

https://bugs-archive.lyrion.org/show_bug.cgi?id=8237

The latest release has broken the plugin, perhaps due to additional brightness checking code in the core.

It might be time to roll this functionality into the main SqueezeCenter code.  Eric Koldinger rewrote AutoDisplay for 7.0:

http://www.koldware.com/SlimStuff/AutoDisplay.zip

I also have several alternate versions of the central algorithim if anyone wants to see them.  One is just a bit of a simplification, the other is the older polled version, and the last is a refinement of my original comparison algorithim that Eric reused but I recast it using modular arithmetic calcuations.

Actually I'll just attach them for posterity.
Comment 7 Daryle Tilroe 2008-05-26 23:34:00 UTC
Created attachment 3371 [details]
Eric Koldinger's original 7.0 port of AutoDisplay
Comment 8 Daryle Tilroe 2008-05-26 23:34:51 UTC
Created attachment 3372 [details]
Some slight mods to Eric's 7.0 port of AutoDisplay
Comment 9 Daryle Tilroe 2008-05-26 23:35:34 UTC
Created attachment 3373 [details]
Daryle's back revision to polling of Eric's 7.0 port of AutoDisplay
Comment 10 Daryle Tilroe 2008-05-26 23:36:27 UTC
Created attachment 3374 [details]
Daryle's revision to modular arithmetic comparisions/calculations of Eric's 7.0 port of AutoDisplay
Comment 11 James Richardson 2008-06-24 09:21:42 UTC
reference bug Bug#: 8237
Comment 12 Chris Owens 2008-06-30 09:23:03 UTC
Dean, could you look at bug 8237 and make a call about whether we want to worry about this bug, or plan to let plugin writers continue doing their thing as we move forward.

Perhaps KDF might consider taking this on as a project.

Also we should bear in mind future products might have complex brightness setting possibilities.
Comment 13 Blackketter Dean 2008-07-18 10:37:16 UTC
Let's leave this to AutoDisplay to fix/update as a plugin.

*** This bug has been marked as a duplicate of bug 8237 ***
Comment 14 Chris Owens 2009-07-31 10:13:43 UTC
Reduce number of active targets for SC