Opened 12 years ago

Closed 12 years ago

#3863 closed patch (fixed)

mythweather causes prebuffer pauses

Reported by: Bill <cizek@…> Owned by: stuartm
Priority: minor Milestone: 0.21
Component: mythweather Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I had mythweather configured to use background updates, and I noticed the following while trying to debug some prebuffer pauses:

2007-08-19 20:50:40.058 Starting update of NDFD-18_Hour
2007-08-19 20:50:40.130 Starting update of NDFD-6_day
2007-08-19 20:50:43.373 NVP: prebuffering pause: AAAAAAAAAAAAAAAAAAAAAAAAAAAaULL
2007-08-19 20:50:44.223 NVP: prebuffering pause: AAAAAAAAAAAAAAALAULAAAAAAAAAAAA
2007-08-19 20:50:45.206 NVP: prebuffering pause: AAAaAALAAAULAAAAAAAAAAAAAAAAAAA
2007-08-19 20:50:47.373 NVP: prebuffering pause: LAAAAAAAUAAAAAAAAAAAAAAAAAAAALA
2007-08-19 20:50:47.660 /mythtv/myth.current/share/mythtv/mythweather/scripts/ndfd18.pl -u ENG -d /home/mythfe/.mythtv/MythWeather/NDFD-18_Hour +41.58,-087.54 has exited
2007-08-19 20:50:49.175 /mythtv/myth.current/share/mythtv/mythweather/scripts/ndfd.pl -u ENG -d /home/mythfe/.mythtv/MythWeather/NDFD-6_day +41.58,-087.54 has exited

The scripts kicked off to refresh the weather data caused enough of a performance hit to affect the current playback.

The attached patch changes mythweather to use 'nice' to reduce the impact of the scripts. This fixed the problem for me.

-Bill

Attachments (1)

niceweather.patch (596 bytes) - added by Bill <cizek@…> 12 years ago.

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Bill <cizek@…>

Attachment: niceweather.patch added

comment:1 Changed 12 years ago by maverik044

Shouldn't mythweather background updates be disabled while watching TV or a video? Doesn't make sense to steal CPU cycles especially for those with frontends with slower CPUs to be quieter and consume less power.

Background updates could lead eventually to weather be shown on the main menu as the clock has been added; therefore disabling background updates all together would be undesirable.

comment:2 Changed 12 years ago by paulh

Component: mythtvmythweather
Owner: changed from Isaac Richards to stuartm

comment:3 Changed 12 years ago by stuartm

Milestone: unknown0.21
Resolution: fixed
Status: newclosed

Closed by [14414]

Note: See TracTickets for help on using tickets.