MMM-OpenWeatherMapForecast Doesn't display imperial units

rslohs

Hello,

Working on OpenWeatherMapForecast and so far have really liked it. One issue, for predicted precipitation I cannot get imperial units to show. I’ve tried changing it within config.js as well as the config file in the module as well. Any suggestions?

{
module: “MMM-OpenWeatherMapForecast”,
header: “Weather”,
position: “bottom_right”,
classes: “default everyone”,
disabled: false,
config: {
apikey: “KEY”,
latitude: “444444”,
longitude: “-4444444”,
iconset: “4c”,
concise: false,
forecastLayout: “table”,
units: “imperial”,
}
},

sdetweil

@rslohs the doc says the module config options is units

from
https://github.com/jharttech/MMM-weatherforecast

rslohs

@sdetweil I’m not quite sure what you’re saying. The ‘Units:“imperial”,’ is incorrect in the code I shared? What should I try differently?

Mykle1

@rslohs

Just a thought: At the top of your config.js file, what are your config units set to?

Example:

var config = {
    address: "0.0.0.0",
    port: 8080,
    ipWhitelist: [], // Set [] to allow all IP addresses
    language: "en",
    timeFormat: 12,
    units: "imperial",

rslohs

@mykle1 thanks for the suggestion, but that is set to imperial as well.

sdetweil

@rslohs its not “Units”, its “units”

this system is case sensitive, every letter matters

rslohs

@sdetweil that’s what I’m using in the config file.

{
module: "MMM-OpenWeatherMapForecast",
header: "Schoolcraft, MI Weather",
position: "top_right",
classes: "default everyone",
disabled: false,
config: {
apikey: "aaaaaaaa",
latitude: "aaaaaaa",
longitude: "aaaaaa",
iconset: "4c",
concise: false,
forecastLayout: "table",
units: "imperial",
}
},

scumbelly

@rslohs

A little late to weigh in but I can verify that imperial setting does not affect the Pop amount. It remains metric. Everything else is imperial.

Sorry.

scumbelly

@rslohs

Guess what? I was trying to troubleshoot an issue in this module re: scale and noticed the .js file has changed wind calculations and the following…

units: {
accumulationRain: {
imperial: “mm”,
metric: “mm”,
“”: “mm”
},
accumulationSnow: {
imperial: “mm”,
metric: “mm”,
“”: “mm”
},
windSpeed: {
imperial: “mph”,
metric: “m/s”,
“”: “m/s”
}
},

So it looks like imperial might work for you now. I’m not going to test because I need the older .js which I had to modify to get some calculations in kph to work .

Cheers

JohnGalt

My instance of this module also displays rainfall in “mm”, even though I have set units = “imperial” for the MagicMirror system as a whole and also in the module configuration.

I see that the Open Weather site has a page specifying “API response parameters with available units of measurement” (https://openweathermap.org/weather-data). There one finds a parameter table specifying Parameter and Description - as well as columns for “Standard”, “Metric” and “Imperial”.

The interesting thing is this: the table specifies temperatures correctly (“Kelvin”, “Celsius”, “Fahrenheit”) but for rain it calls out “mm”, “mm”, “mm” - or other variants for different fields, e.g.: “mm/hr” and “mm/hour”.

As such, I am inclined to just give up and accept the module’s behavior as a quirk of how Open Weather parses what it is getting from the NWS - but I thought i would see if anyone has a better idea.

kayakbabe

@JohnGalt have you tried MMM-OpenWeatherForecast instead? It seems to work better for me adn it uses the OpenWeather Map api too.

JohnGalt

@kayakbabe - Thanks for the tip. Can you tell me which one you use? A quick web search shows several modules with the same name. These two look like top contenders:

https://github.com/jclarke0000/MMM-OpenWeatherForecast
https://github.com/themaxgoldman/MMM-OpenWeatherForecast

JohnGalt

@kayakbabe PS: another issue with MMM-OpenWeatherMapForecast is that it doesn’t respect the “disabled: true” configuration. I had seen this before and ignored it since the module is active, but when I went in to disable it in preparation for trying one of the others, I am reminded that I cannot just disable it. I CAN put " /* and */ " around the whole thing of course, but that is just clunky, and more importantly may signal more serious coding issues.

JohnGalt

@kayakbabe FWIW, i installed the jclarke0000 module, and so far it looks pretty good - thanks again!

sdetweil

@JohnGalt disabled is an mm config option, not a module option. it goes not to the module line, outside the module config:{} section

here is the doc on all the mm config options for a module.

these are separate from the module config options inside the config:{} section

https://docs.magicmirror.builders/modules/configuration.html#example

JohnGalt

@sdetweil - Thanks Sam - I guess I was and am confused.

Below are two code snips showing the subject module and one other. Both are designated to be disabled with the “disabled: true” instruction outside the config section. The top one still operates, while the bottom one [and every other module I have edited this way] is disabled. So, it’s not a big deal and I will keep an eye out for how I do things. Thanks again.

{module: "MMM-OpenWeatherMapForecast", // https://github.com/MarcLandis/MMM-OpenWeatherMapForecast NOTE: See also https://github.com/sdetweil/MMM-OpenWeatherMapForecast
  disabled: true,
  header: "", // "Weather"
  position: "top_right",
  classes: "default everyone",
  disabled: false,
  config: {
    apikey: "xxxxxxxxxxxxx",
------------------------------------------------------------------------------- 
{module: "MMM-Temperature", // https://github.com/Tom-Hirschberger/MMM-Temperature
  disabled: true,
  header: "Office Temp + Humidity",
  position: "top_right", // bottom_right
  config: {
   sensors: [

sdetweil

@JohnGalt I need to see the rest in between

top one has disabled twice, true and false

JohnGalt

@sdetweil - Ah crap, having both disabled: true AND disabled: false would do it!! :) I’m sure the system would execute the last command. Thanks for putting a second set of eyes on it.

sdetweil

@JohnGalt yes, last occurrence wins

kayakbabe

@JohnGalt
I see you got your module choice working.
jclarke0000 is the one I’m using.