Weatherunderground - currently - hourly - daily - configurable

RedNax

@Shockwave Well, it would seem WU is rather ‘loose’ with it’s API implementation…

In your payload the full warning is in the message property. The Wunderground module expects is in the description property (only based on my observation of a single payload by my local station :S ). Ties in with your observation, which is always nice :)

Watch this space for a fix…

UPDATE: fix implemented… The module examines description and message lengths and uses the longest one… Have a go! :)

Shockwave

@RedNax It works! Thanks for fixing that so fast. Now I need to politely bug MichMich about the size of the alert box since our weather service is so verbose. :)

Pauliolio

@RedNax

That’s a handy test - Many thanks.
When I request:
http://api.wunderground.com/api/[my key]/conditions/forecast10day/astronomy/alerts/lang:EN/q/pws:IBATTLE3.json

I get a full set of data back.
The module is still getting stuck on ‘Loading’ on the mirror though.
My config is:

{
    module: 'MMM-WunderGround',
    position: 'top_right',
    config:  {
            apikey: '[my key]',
            pws: 'IBATTLE3;loctype:1',
            lang: 'EN',
            fctext: '1',
            fcdaycount: "5",
            fcdaystart: "0",
            hourly: '1',
            hourlyinterval: "3",
            hourlycount: "2",
            alerttime: 10000,
            alerttruncatestring: "english:"
    }
},

Can you see anything wrong there please?

Thank you.

Note from admin: Please use Markdown on code snippets for easier reading!

Pauliolio

@RedNax

This may explain the issue to you hopefully. I thought I’d given you this info before, but it seems not.

In the console I get an error:
MMM-WunderGround.js:407 Uncaught TypeError: Cannot read property 'length' of undefined

The relevant code is:

	processWeather: function(data) {
		this.alerttext = "";
		this.alertmsg = "";
		for (var i = 0, count = data.alerts.length; i < count; i++) {

Any idea why the array would be undefined?

Many thanks.

Note from admin: Please use Markdown on code snippets for easier reading!

RedNax

@Pauliolio The issue is with the config. try the following change:

pws: 'pws:IBATTLE3',

yo-less

Hi @RedNax,
thanks for the excellent module! I seem to have some difficulty having night time icons displayed. I’m not sure what I’m doing wrong as this is what my module looks like at 1 am in the morning:

Using the API in my browser window I get the following two lines:

“icon”:“clear”,
“icon_url”:“http://icons.wxug.com/i/c/k/nt_clear.gif”

It seems to me that I’m getting the daytime “clear” condition icon, maybe due to the ambiguous icon declaration, anything I can do to change icons like this one to nighttime icons?

RedNax

Easiest way to do this is to define another icontable for nighttime use and switching to it after sunset…

When i have some time, i’ll look into it…

UPDATE: Who am i kidding ;) Nighttime icons now included…

murattyilmaz

Where is the github url or download url of these module? Can’t find it.

KirAsh4

@murattyilmaz,

You need to look under the Showcase category to find the actual link. This category here is the Troubleshooting side of those modules.

https://forum.magicmirror.builders/topic/128/weatherunderground-currently-hourly-daily-configurable

Pauliolio

@RedNax

Hi - I’d already tried that unfortunately. Still gets stuck at the same point in the code & displaying ‘Loading’ on the screen.

Would wrapping that for loop in an if that checks whether data.alerts is actually defined or a try block help do you think?