View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0014853 | MMW 5 | Now Playing | public | 2018-05-31 20:30 | 2020-07-14 12:18 |
| Reporter | Ludek | Assigned To | |||
| Priority | urgent | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 5.0 | ||||
| Target Version | 5.0 | Fixed in Version | 5.0 | ||
| Summary | 0014853: Very high CPU utilization during playback in Now Playing > 'Large Lyrics' layout | ||||
| Description | When I play a track with lyrics and the view is Now Playing > 'Large Lyrics' layout then CPU utilization is very high (30% -- which are two cores on my i7) | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 2128 | ||||
|
|
Debugging a little and I found that the CSS class 'veryLargeShadowedText' is causing this with combination of the background image animation (background image scrolling during playback). It took me a while to find this, because the issue appears even for artists without the background image (and unfortunately I tested on such an artist at first) |
|
|
Yes, I've seen this in the past, but haven't decided what to do about it -- the current implementation looks quite good and it's just a bit too CPU intensive. Leaving this open to decide later whether some kind of a change is needed. |
|
|
Isn't the solution just disabling the background image animation scrolling for this layout ? It looks like quite severe issue to me, because it can drain the user's battery while playing tracks. |
|
|
It's actually supposed to be disabled, it's the default for this view (and I've just hardcoded it, to be sure). However, there's a problem, after deleting persistent.json (for a fresh start) and switching to 'Lyrics' NP view, BackgroundImage.restoreState() is called and backgroundType is set there to 'albumdark'. There shouldn't be any value in restoreState provided, since we started with a fresh persistent.json. |
|
|
Fixed |
|
|
Verified 2129 |
|
|
re tested 2260 CPU utilization is normal now. |
