Weather 2 Wishlist

[whurlston]

You are probably not doing it wrong. I might have messed it up attempting to get both alerts and warning working. I'll try to take a look at this on Monday or Tuesday and see if I can fix it.

[whurlston]

I'm aware of the alerts issue. I was able to fix one type of alert by modifying Plugins\Weather\Config\sources.xml and replacing the "AlertNodes" tag with the following:

		<AlertNodes>
			<Node>//div[@class='panel-body-lt']/div[@class='panel-body-rt']/div[@class='panel-body']/div[@class='panel-body-content']/div[@class='tab-panel']/div[@class='content awx-story']</Node>
		</AlertNodes>

This however AFAIK will only give you alerts and not warnings. I haven't had any warnings to test parsing those.

Some changes to the plugin are on my todo list that will allow for better parsing but I'm not sure when I will get to it.

I just retested this and I am indeed getting the alerts. It only works if this is the only <Node> line in <AlertNodes> though. If it isn't working or if you experience a severe weather warning, please zip and post the following file:

Plugins\Weather\config\[Some long GUID]\alert_source.html

[Snooze]

Yes that fixed the alert. I must have messed up the XML the first time I tried it. Thanks for getting back to this.

[mikeh49]

@whurlston:

Tried the fix, but doesn't work for me. The alert-source html is attached, and I think I can see why it doesn't work. The alert link doesn't go the alert text, but instead to a "story" about the alert. Compare to the small craft advisory (assuming it is still there when you get to this). Not sure what can be done about something like this.

[whurlston]

@whurlston:

Tried the fix, but doesn't work for me. The alert-source html is attached, and I think I can see why it doesn't work. The alert link doesn't go the alert text, but instead to a "story" about the alert. Compare to the small craft advisory (assuming it is still there when you get to this). Not sure what can be done about something like this.

I see the problem. The alert you are getting is under "/div[@class='info']" instead of "/div[@class='content awx-story']". Adding this as a second node will not work. It looks like the plugin was coded when accuweather was supplying all nodes even if they were empty but they are no longer doing that. So the plugin is erroring when one of the alert nodes is missing. I'll see what I can do to fix that.

[mikeh49]

@whurlston:
Here are a couple more alert.html files from today. The one for Sheboygan does not work in Weather (cannot parse error), but the one for Minneapolis does. Note that the advisory text in the one for Minneapolis is "expanded" when the file is opened in a broweser, but the one for Sheboygan is not. I don't know if this helps, but it is more data.