Opened 13 years ago

Closed 13 years ago

#1300 closed defect (fixed)

mythweather: number of Doppler radar images

Reported by: amcnabb@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythweather Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

At least for the locale USUT0208 (Provo, UT), there are only 5 Doppler radar images on image.weather.com. The URL "http://image.weather.com/looper/archive/us_slc_closeradar_large_usen/6L.jpg" does not return an image, though 1L.jpg through 5L.jpg work fine. Probably the for loop in weather.cpp should cycle between 1 and 5 (instead of 1 and 6 as at present). However, if other locales have more images, it might be necessary to glean the number of images from the javascript on the referring page.

Change History (5)

comment:1 Changed 13 years ago by anonymous

Version: head

comment:2 Changed 13 years ago by Isaac Richards

Resolution: worksforme
Status: newclosed

Not sure what the bug is, here. That it's trying to download something that it doesn't need to? Doesn't really hurt anything - mythweather won't display an empty/non-existant image, so there's nothing functionally wrong that I can see. Tested with the given locale.

comment:3 Changed 13 years ago by anonymous

Resolution: worksforme
Status: closedreopened

When mythweather can't download an image, it displays a black screen with a message saying, "Map Unavailable." So if an image doesn't exist, you get little bits of radar video separated by a jarring black frame.

comment:4 Changed 13 years ago by anonymous

I agree. I have noticed this problem over the last few weeks as well. Even though there is no real problem, per se, it really makes it difficult to view the image when the black frmae pops up!

comment:5 Changed 13 years ago by cpinkham

Resolution: fixed
Status: reopenedclosed

(In [9184]) Parse the number of Radar images from the html source instead of assuming 6 images are in the radar loop. If the number cannot be determined by looking at the code, it defaults to 5 images since that is the maximum number of images some users are seeing now. Closes #1300.

Note: See TracTickets for help on using tickets.