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

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.