Weather Forecast: showRainAmount displays in mm even when config.units='imperial'

[fjnorb]

Hello. Brand new MM2 user, I love it. Thank you for taking the time to make it easy to install and configure. I spent weeks last year fighting with Conky to do my own thing and I had this running immediately after installation.

I have a small issue, however, that I hope someone has solved. Being in the United States, I have units set to ‘imperial’ in my config, and both the Current Weather and Weather Forecast are using imperial units for temperature and other things. But I enabled showRainAmount, and the rain amounts are showing in millimeters.

Is this a limitation with OpenWeatherMap, a missing conversion in the Weather Forecast module, or a mistake on my part?

Thanks for any and all assistance.

[MichMich]

It seems the “mm” part is hard coded:
https://github.com/MichMich/MagicMirror/blob/master/modules/default/weatherforecast/weatherforecast.js#L153

Does it really show the amount in mm? or does it just show the wrong unit?

[MichMich]

Hmmm… it seems OpenWeatherMap returns the data in MM no matter which what we request.
What unit would be preferred? Inch? What notation would you recommend?

[strawberry 3.141]

@MichMich wikipedia tells inch for rain measurements https://en.wikipedia.org/wiki/Rain#Measurement

if(config.units === "imperial"{
  rainCell.innerHTML = forecast.rain / 25.4 + " in";
}

probably need parseFloat() for the data if its a string

[fjnorb]

It looks like it is returning millimeters. For instance, it says on Tuesday we’re going to get 15mm of rain, that’s about a half an inch, which is much more reasonable than the notion that we’d get 15 inches (381mm) of rain in one day. That sort of thing would be in the news feed, lol.

[MichMich]

I pushed an update to the develop branch. Please give it a try.

[fjnorb]

Can you give me a quick idea of how to do that please? Installation was so simple I don’t know how.

[strawberry 3.141]

@fjnorb

cd ~/MagicMirror
git fetch
git checkout develop
npm install

[fjnorb]

Looks great! Appreciate the quick fix and quick responses, wish more FOSS projects were this professional and responsive.