search

MatthewFlamm / nwsradar

Home assistant custom camera platform for NWS radar loops
MIT License
20 stars 2 forks source link

Make it clearer to user when no radar images are available for enhanced and Mosaic styles #4

Open jsiemonski opened 4 years ago

jsiemonski commented 4 years ago

I upgraded from 0.4.0 today, and seem to just have a black screen.

MatthewFlamm commented 4 years ago

0.4.0 had a fatal bug, with was fixed in 0.4.1. Are you sure your are on 0.4.1 and restarted your home assistant? Which radar style are you using? Have you checked whether the radar is online on the website?

MatthewFlamm commented 4 years ago

Also are there any errors in your logs?

jsiemonski commented 4 years ago

3188366B-8C1B-44BB-8152-1F6FFD6320C3 1F2320BD-C0C2-4594-AC60-FA3F0CAB3D33

I will check the logs when I get a chance.

MatthewFlamm commented 4 years ago

Looks okay on my system using standard style, although you've got some serious storms.

Screenshot_20200430-185204

jsiemonski commented 4 years ago

I have it in Enhanced, does that matter? I’ll try a reboot when I get home.

MatthewFlamm commented 4 years ago

This is not a problem with the new version. There are no radar files available right now, see below where there would be a list of images normally. I'm thinking that either an error should be logged or the entity should be marked unavailable, or both. Or maybe instead of being black, it shows 'radar not available'.

On a positive side, the NWS website shows no storms in the enhanced view while my package makes it black in this case.

I'm going to keep this open until there is better error handling after I think about the best way.

Screenshot_20200430-200952

jsiemonski commented 4 years ago

Whew.. just happened to coincide with my upgrade. Thanks for checking it out! I was just digging through my logs..

Sent from my iPhone

On Apr 30, 2020, at 8:14 PM, MatthewFlamm notifications@github.com wrote:

 This is not a problem with the new version. There are no radar files available right now, see below where there would be a list of images normally. I'm thinking that either an error should be logged or the entity should be marked unavailable, or both. Or maybe instead of being black, it shows 'radar not available'.

On a positive side, the NWS website shows no storms in the enhanced view while my package makes it black in this case.

I'm going to keep this open until there is better error handling after I think about the best way.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

MatthewFlamm commented 4 years ago

I'm also noticing the standard style I posted is showing a radar from 12 hours before I loaded it. NWS website shows the same. I should also check the times to avoid using stale images (this will only work for enhanced style probably).