Bug 9218 - Transporter FW62 has broken volume control screen...
: Transporter FW62 has broken volume control screen...
Status: RESOLVED INVALID
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: unspecified
: PC Other
: -- normal (vote)
: ---
Assigned To: Matt Wise
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-19 19:08 UTC by Matt Wise
Modified: 2009-09-08 09:19 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
heres what it looks like first... (651.62 KB, image/jpeg)
2008-08-19 19:08 UTC, Matt Wise
Details
here's what it looks like after a half second... (650.87 KB, image/jpeg)
2008-08-19 19:09 UTC, Matt Wise
Details
fixed MusicInfoSCR plugin (24.61 KB, application/zip)
2008-08-25 02:00 UTC, Michael Herger
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Wise 2008-08-19 19:08:58 UTC
Created attachment 3834 [details]
heres what it looks like first... 

The volume control screen is broken when you immediately change it. It fixes itself after you let it sit for a half second. Attached are two example photos. I'm running SC (boom) 22765, so thats why I marked this bug private. Not sure if its a general 7.2 problem or not.
Comment 1 Matt Wise 2008-08-19 19:09:19 UTC
Created attachment 3835 [details]
here's what it looks like after a half second...
Comment 2 Blackketter Dean 2008-08-19 20:01:06 UTC
Triode: Could this be related to change 22669
Comment 3 Michael Herger 2008-08-19 23:02:11 UTC
Are you using my MusicInfoSCR screensaver for the alternative volume display? Triode mentioned on campfire it might be broken.
Comment 4 Matt Wise 2008-08-20 07:37:28 UTC
I do have it installed... but I wasn't actually using the screen. Do I automatically get the different Volume screen with it? 
Comment 5 Michael Herger 2008-08-20 07:49:01 UTC
Yes, I think default for the alternative volume screen is on.
Comment 6 Adrian Smith 2008-08-20 10:44:17 UTC
So the normal volume screen does show dBs for the transporter, but the display you have here is from MIS - its the custom volume control.

I don't believe the recent volume display changes will impact this as they are now an optimisation of the lines function for input bar.  Just tried MIS and it appears to work ok.

Can you disable the custom volume display in the Music Info settings Matt and see whether it looks ok?  If so we need to debug why you get this display but it is potentially MIS only (and doesn't happen for me!)
Comment 7 Blackketter Dean 2008-08-21 23:57:05 UTC
Ping matt.  Still an issue?
Comment 8 Michael Herger 2008-08-22 00:05:50 UTC
FWIW: I see it now with my MIS screensaver too. But I only see it when pressing the Volume front panel - using the remote is ok. Maybe still a FW issue. But surely not a Boom bug.
Comment 9 Felix Mueller 2008-08-22 02:44:57 UTC
With player-display logging on, I get this when pressing TR's front panel volume button:

[08-08-22 11:39:49.2035] Slim::Display::Display::curLines (487) lines Plugins::MusicInfoSCR::Plugin::volumeLines [Slim::Buttons::Common::pushModeLeft(962)]
[08-08-22 11:39:49.2036] Slim::Display::Display::curLines (488) {
  fonts   => {
               "graphic-320x32" => {
                     line => ["standard.1", "MIScrVolumeSB2.2"],
                     overlay => [undef, "MIScrVolumeSB2.2"],
                   },
             },
  line    => ["Volume", "\1ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_`a"],
  overlay => [undef, "-98413920.0\2"],
}

And after a second it clears up:

[08-08-22 11:39:50.9093] Slim::Display::Display::curLines (487) lines Plugins::MusicInfoSCR::Plugin::volumeLines [Slim::Display::Display::update, (eval)(198)]
[08-08-22 11:39:50.9134] Slim::Display::Display::curLines (488) {
  fonts   => {
               "graphic-320x32" => {
                     line => ["standard.1", "MIScrVolumeSB2.2"],
                     overlay => [undef, "MIScrVolumeSB2.2"],
                   },
             },
  line    => ["Volume", "\1ABCDEFGHIJKLMNOP"],
  overlay => [undef, "-27.5\2"],
}

So, yes, I guess, it is an uninitialized variable somewhere.
Comment 10 Blackketter Dean 2008-08-22 12:50:20 UTC
Ok, is this something we need to fix in the SC release or is it a third-party plugin issue?
Comment 11 Michael Herger 2008-08-22 13:25:53 UTC
I'll take some time to fix my plugin. IMHO it doesn't show anywhere else.
Comment 12 Michael Herger 2008-08-25 02:00:26 UTC
Created attachment 3876 [details]
fixed MusicInfoSCR plugin

Matt - fwiw: my plugin needed a fix. Triode changed the way custom volume functions are called a while back. I missed that change. This version should fix this issue. Please give it a try.