Bug 9329 - Alarm: Make alarm volume and duration configurable per alarm
: Alarm: Make alarm volume and duration configurable per alarm
Status: REOPENED
Product: Logitech Media Server
Classification: Unclassified
Component: Alarm
: 7.2
: PC SuSE Linux
: -- enhancement with 11 votes (vote)
: Future
Assigned To: Max Spicer
:
Depends on: 3170
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-29 05:04 UTC by slimpy
Modified: 2012-02-09 05:41 UTC (History)
13 users (show)

See Also:
Category: Feature


Attachments
Implement API and CLI for per-alarm timeout plus other bits (12.22 KB, patch)
2009-04-12 06:02 UTC, Max Spicer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description slimpy 2008-08-29 05:04:19 UTC
With kdf's Extended Alarm Plugin now deprecated I would like to see some of the plugin's useful features in the new alarm.

Alarm volume and alarm duration should be configurable per alarm and not just globally for all alarms.

If different alarms on one Squeezebox are used by different people it can be preferrable to have different alarm volumes on different alarms.

A light sleeper needs a lower volume to wake up vs. a sound sleeper who needs a loud alarm to get out of bed.

Alarm duration is also likely to depend on the persons habits.
Someone who stays in bed for some time after waking up needs a longer duration than someone who gets up instantly when the alarm goes off.
Comment 1 Max Spicer 2008-09-02 07:42:02 UTC
Volume already is configurable per-alarm, but there's no ui for it.  You can set an alarm's volume individually by editing it in the prefs file or using the cli.  The ui used to have per-alarm volume settings, but it made for a lot of settings.  Sounds like a pref to enable advanced alarm settings might be called for, but it's a bit of a ui cludge.

Duration per-alarm is not possible atm.
Comment 2 Chris Owens 2008-09-15 09:18:48 UTC
Dean, could you help target this?  Or should we just let it alone to accumulate votes and comments?
Comment 3 James Richardson 2008-09-29 20:11:02 UTC
*** Bug 9609 has been marked as a duplicate of this bug. ***
Comment 4 Tim Sharpe 2008-11-01 13:01:23 UTC
I use KDF's extended alarm plugin as well and keep having to hack at it to keep it running.  I have 2 SB3's (one in bedroom and one attached to stereo), each of which alarms at different times each day.  I use different volume settings for each SB and each alarm has a different duration (some end after various set amounts of time, some don't end at all).

Until the Alarm Clock supports individual volume and duration (specific duration or no specific timeout), the Alarm Clock really isn't a replacement for KDF's plugin.  KDF did such a nice job with functionality, it's shame that the level of functionalty he created hasn't made it into the only other game in town.
Comment 5 Hal Forman 2009-02-28 16:15:49 UTC
I would very much like to be able to configure alarm volume per alarm.  Duration would be helpful too.
Comment 6 chad_m_meyers 2009-03-07 09:45:47 UTC
volume configurable per alarm would be very useful.  thanks.
Comment 7 Tim Sharpe 2009-04-11 12:25:42 UTC
Is there any idea as to when this might be dealt with?  As there is no real substitute for the now-deprecated KDF Extended Alarm plugin, it strikes me that this is a bit bigger than a "future enhancement". 

While I realize that your priorities center around playing media and such, this is a serious functionality issue, that impacts the usefulness of the device to a potential audience beyond audio enthusiasts.  It seems to go toward the heart of device utility.

I'd just like some idea of when I can stop babying the old KDF plugin to keep it working.  Months? Years? Never?  Just setting a target version for inclusion would be a blessing.

Thanks!  Aside from this one issue, I'm extremely satsified with the SB#/Squeezecenter platform.  Our household literally "lives" on it for access to Public Radio (we're in a fringe area without receivable "over the air" stations.  Running on a old Celeron machine with Debian Lenny, it's a solid, stable delivery platform.
Comment 8 Max Spicer 2009-04-12 06:02:21 UTC
Created attachment 5103 [details]
Implement API and CLI for per-alarm timeout plus other bits

Dean decided originally that allowing per-alarm volume in the ui made things too complex.  I'm not proposing to change this decision now.  However, the alarm code has always supported per-alarm volumes and these could be set via plugin code or the CLI.  The attached patch also implements per-alarm timeout settings, again settable through code or the API.  See the docs in Slim::Utils::Alarm.pm for details, and also the CLI docs.  None of this is exposed in the default user interface.

This patch also adds the ability to query whether an alarm uses the default volume/timeout via the CLI by adding to new tagged parameters to the alarms query - usesDefaultVolume and usesDefaultTimeout.  Additionally, it allows you to change an alarm back to using the default volume/timeout via the cli alarm command by setting the timeout or volume parameter to -.  Again, see CLI docs for more info.

I'd like to check this patch into 7.4, but am unsure of the status of the 7.4 branch.  Please advise if it's safe to do this.

Tim: rather than spend time keeping KDF's deprecated alarm plugin going, I'd advise you consider writing a plugin to expose this new functionality in the user interface.  Alternately, you could use the functionality via the CLI or possibly the http interface.

We could consider adding these into the standard ui via, possibly enabled via a 'enable advanced alarm settings' pref.  However, I'm going to leave that decision to Dean et al.  An alternative may be to modify the alarm code to allow plugins to extend it with the addition of extra sub-menus.  This may already be possible with the Controller interface to the alarms.
Comment 9 Tim Sharpe 2009-04-12 09:16:06 UTC
(In reply to comment #8)
I'd be happy to if I were a good enough programmer.  I barely understand Perl and KDF's plugin well enough to deal with the problems that crop up with each SC update (mainly keeping alarms in the schedule).

Otherwise, I'd be happy to help and would have already submitted something...

Thanks!

> Created an attachment (id=5103) [details]
> Implement API and CLI for per-alarm timeout plus other bits
> 
> Dean decided originally that allowing per-alarm volume in the ui made things
> too complex.  I'm not proposing to change this decision now.  However, the
> alarm code has always supported per-alarm volumes and these could be set via
> plugin code or the CLI.  The attached patch also implements per-alarm timeout
> settings, again settable through code or the API.  See the docs in
> Slim::Utils::Alarm.pm for details, and also the CLI docs.  None of this is
> exposed in the default user interface.
> 
> This patch also adds the ability to query whether an alarm uses the default
> volume/timeout via the CLI by adding to new tagged parameters to the alarms
> query - usesDefaultVolume and usesDefaultTimeout.  Additionally, it allows you
> to change an alarm back to using the default volume/timeout via the cli alarm
> command by setting the timeout or volume parameter to -.  Again, see CLI docs
> for more info.
> 
> I'd like to check this patch into 7.4, but am unsure of the status of the 7.4
> branch.  Please advise if it's safe to do this.
> 
> Tim: rather than spend time keeping KDF's deprecated alarm plugin going, I'd
> advise you consider writing a plugin to expose this new functionality in the
> user interface.  Alternately, you could use the functionality via the CLI or
> possibly the http interface.
> 
> We could consider adding these into the standard ui via, possibly enabled via a
> 'enable advanced alarm settings' pref.  However, I'm going to leave that
> decision to Dean et al.  An alternative may be to modify the alarm code to
> allow plugins to extend it with the addition of extra sub-menus.  This may
> already be possible with the Controller interface to the alarms.
Comment 10 James Richardson 2009-05-05 07:30:31 UTC

*** This bug has been marked as a duplicate of bug 3170 ***
Comment 11 Max Spicer 2009-05-05 07:43:55 UTC
This is not a duplicate of bug 3170.
Comment 12 Max Spicer 2009-05-05 07:45:16 UTC
*** Bug 11971 has been marked as a duplicate of this bug. ***
Comment 13 James Richardson 2009-11-12 12:44:04 UTC

*** This bug has been marked as a duplicate of bug 3170 ***
Comment 14 Kyle K. 2009-11-12 12:49:06 UTC
This bug needs to be reopened - it is not a duplicate of 3170.

Bug 3170 deals with a single alarm volume being increased over time.

This bug deals with multiple alarms having different volume levels.

Please reopen and remove duplicate tag.
Comment 15 James Richardson 2009-11-12 13:13:47 UTC
Well stated, removing duplicate tag

Max: is your patch working should we check it into 7.5 to get some eyes on it?
Comment 16 Max Spicer 2009-11-13 01:42:00 UTC
I believe my patch is working.  It certainly was when I wrote it, and I've had it in my local copy since then.  I don't use this feature myself, however, so this only tells you that it doesn't cause the server to obviously break.  Should be fine to check into 7.5 if you want it.