Bug 7608 - press and hold volume buttons doesn't work on boom
: press and hold volume buttons doesn't work on boom
Status: CLOSED FIXED
Product: SB Boom
Classification: Unclassified
Component: Setup
: unspecified
: Macintosh Other
: -- normal (vote)
: ---
Assigned To: Felix Mueller
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-03-24 17:04 UTC by Blackketter Dean
Modified: 2008-12-15 11:56 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2008-03-24 17:04:30 UTC
I get this error instead:

[17:02:04.1998] Slim::Hardware::IR::executeButton (991) Error: Subroutine for irCode: [arrow_down.hold] does not exist!


also, when the volume indicator is still on screen, you should be able to turn the knob to adjust hte volume, like transporter and jive.
Comment 1 Blackketter Dean 2008-03-25 16:30:05 UTC
*** Bug 7614 has been marked as a duplicate of this bug. ***
Comment 2 KDF 2008-04-04 23:12:04 UTC
would also be nice to have the knob work the same as the transporter and the controller, in that once you are in volume mode, the scroll will also affect the bar.
Comment 3 Adrian Smith 2008-04-05 16:22:41 UTC
Agreed.

Felix - I've been playing with this, looks to me that the knob sends both the knob ir codes and arrow ir codes.  It would be nice if we could only send the knob ir codes

This is what I get from dumping the ir codes rotating it one way

0002005b - 18082.309
0001005a - 18082.391
0002005a - 18082.405
0001005a - 18082.437
0002005a - 18082.448
0001005a - 18082.469
0002005a - 18082.474

Reason for this is that volume requires the direction to be swapped in comparison to other lists, so I think the FrontPanel mapping should be to knob_left and knob_right we can then map to the appropriate function in Default.map and reverse the direction for the volume mode (which is broken in Default.map at present...)
Comment 4 Adrian Smith 2008-04-05 16:25:56 UTC
Also - would the volume buttons be able to repeat on the player or does this need to be added in the server.  There's nothing in the server for this at present but could be done, just wondered if it was as easy in the player.
Comment 5 Blackketter Dean 2008-04-05 16:28:28 UTC
Something weird is going on here, see bug 7593.  Text entry is also broken, but this stuff all works right on Transporter, though the knob is a lot different on that device.
Comment 6 Adrian Smith 2008-04-05 16:41:46 UTC
Are there any plans to emulate the knob on Transporter in firmware?  That sends absolute values so won't suffer from large round trips the way just sending up/down codes may (assuming you spin the knob fast).

I'd definately get rid of one set of the ir codes and then play with the mappings for the current approach though.
Comment 7 Felix Mueller 2008-04-06 03:05:20 UTC
Added code in firmware to make volume up/down and fwd/rev repeating (with acceleration). (rev4053)

No knob volume yet. (Needs to be done in firmware for offline mode and in SC7 when connected.)

The knob only sends knob events. What you are seeing here are knob_up (press/release) and knob_down (press/release) events. Well, pressed and released are not the correct terms, I know.

0002005b - 18082.309 -- knob_down / released
0001005a - 18082.391 -- knob_up / pressed
0002005a - 18082.405 -- knob_up / released

I changed the behavior so you now only should see knob_left and knob_right events. (rev4054)

0001005a -- knob_left
0001005b -- knob_right

Let me know if that works for you.
Comment 8 Adrian Smith 2008-04-06 03:30:37 UTC
Ah I understand now - I didn't see the previous entries in the Front_Planel.ir

So my proposal is that in Front_Panel.ir we map these codes to knob_right and knob_left

We then put the appropriate additional mapping into the relavent modes in Default.map to use these - this will allow the mapping to be reversed for input modes like volume and text where we want knob_right to be the opposite to arrow_down.

I can look at the knob volume in SC as I almost had that working yesterday, just the wrong way round!

Is there a binary of this code available - I can only see the SC repository and its not in there..
Comment 9 Felix Mueller 2008-04-06 03:40:46 UTC
I can send it to you offline. It's still version 2, so you will need to use an old remote and press and hold brightness to get it loaded. What email should I use?
Comment 10 KDF 2008-04-07 14:35:32 UTC
*** Bug 7759 has been marked as a duplicate of this bug. ***
Comment 11 Blackketter Dean 2008-05-02 12:01:06 UTC
Seems to be fixed.
Comment 12 James Richardson 2008-12-15 11:56:17 UTC
This bug has been fixed in the latest release of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.