Changing units from 'metric' to 'imperial' breaks currentweather module

[vaju0503]

Just look at the history of the change that was made in the other branch… it’s in modules/default/currentweather the script you need to update is currentweather.js

I just used an SFTP connection to get into my pi, went to the directory and uploaded the new script. You have to change the following in the script lines 410 and 411. Take note in the case sensitive variable of windChillInF it was changed from windchillinF. You could do it from terminal too using nano or whatever your favorite editor is. That’s what I ended up doing until this gets merged up into the main pull.

var windChillInF = Math.round(35.74+0.6215*tempInF35.75*Math.pow(windInMph,0.16)+0.4275*tempInF*Math.pow(windInMph,0.16));
var windChillInC = (windChillInF - 32) * (5/9);

Obviously after you fix it you’ll have to restart MM or just do a reboot and it should fix itself.

[dplawrance]

Thank you!

That was so simple, and it works! I tried (and failed) to switch over to the the develop branch, learned a lot more about git than I knew, but not enough.

[ryanvox]

My current weather is still broke after update. Tried git checkout and it says my branch is up to date with origin/master. When I change back to metric from Imperial it kind of works. It only gives me temp and tells me its snowing or raining or whatever. Its missing sunset time, wind speed, etc. Also changed the script as stated above in modules/default/currentweather. rebooted and still no luck.

EDIT
Was able to get the Imperial system working but still missing wind speed, sunset/sunrise time

[dorfman2]

Some days my current Weather module works, some days it doesn’t. My mirror is set to restart daily at 1am. It’s been working the past 2 days, but hadn’t worked the prior week. I haven’t had time to look into it, all I did was update it to 2.3.1 on a raspberry pi 3 B+.