MMM-MyWeather not working with Southern Hemisphere co-ordinates

Jolt02

Hi all!

Need help with an odd one. Is anyone else running into issues using MMM-MyWeather for Southern Hemisphere co-ordinates? The module gets stuck on “loading” if lat has a negative value, but works fine for negative value in longitude (eg New York with lat = 40.69 and lon = -74.25).

The response from the API as provided in the logs is fine in both instances, but it just looks like the module cant parse the json correctly and i can’t figure out why.

Sample of my config below.

modules: [
  {
    module: 'MMM-MyWeather',
    position: 'top_right',
    config: {
      apikey: 'xxxxxxxxxxxxx', 
      lat: "-22.27", // 
      lon: "133.77" , // 
      hourly: '1',
      fctext: '1',
      fcdaycount: "5",
      fcdaystart: "0",
      hourlyinterval: "3",
      hourlycount: "2",
      UseCardinals: 0,
      layout: "horizontal",
      debug : 1
    }
  },
]

sdetweil

@jolt02 remove the quotes from all the numbers

Jolt02

@sdetweil that’s not the issue though. using lat = “22.27” and “133.77” work even if the rest of numbers are in quotes.

the code above were copied and pasted the config from the module documentation. i’ve just changed the co-ordinates to be southern hemisphere co-ordinates

sdetweil

@jolt02 said in MMM-MyWeather not working with Southern Hemisphere co-ordinates:

  fcdaycount: "5",
  fcdaystart: "0",
  hourlyinterval: "3",
  hourlycount: "2",

but i don’t care what the author posted… those are strings, that need to be converted to numbers…

anyhow

Jolt02

@sdetweil 🤷‍♂️ that’s how they built the module and i’m not familiar enough with JS to go through and fix it. Thanks for trying anyways.

for what it’s worth, I ended up learning to use the console more and figured out something in the moon direction logic was broken because it was looking for there was a condition for lat >= 0. I’ve just gone through and deleted all references to moon phases in the MMM-MyWeather.js file and it’s working fine now.

probably not the most elegant of solutions, but at least it gives me way more information than “loading…”

sdetweil

@jolt02 said in MMM-MyWeather not working with Southern Hemisphere co-ordinates:

a condition for lat >= 0

so, to add to the prior discussion

if the user had coded lat:“-16” in config.js

the comparison would always be true comparing a string to a 0 (all ‘string’ objects would be non-zero, regardless of their content)

harney

try taking out the Lon And Lat and use a PWS like for me would be… Perth (IPERTH921). This usually happens when the site can’t match a co-ordinate to a location.

I have tried various modules that require Lon and Lat or area and found this happens a lot ‘down under’ as there seems to be less active locations than in the northern hemisphere.

for my mirror I used default weatherforecast and MMM-OpenmapWeather . Both of these use Open Weather map that has locations rather than co-ordinates.

MMM-MyWeather also uses locations (PWS)

Jolt02

@harney hello from Perth as well! how are you finding the accuracy of the OpenWeather data? I was using that initially, but found that it was telling me something different to the results from Google.

i used lat long in OpenWeather though and didnt try using the PWS so that might be the difference

Jolt02

@sdetweil from my very basic understanding of how it was developed, i think it was stored it as a string so that it could be concatenated? again, this is based on 5 minutes of Google-fu and no understanding of JS here

sdetweil

@jolt02 js will convert in either direction automatically, except if u ask, is this a number or a string

temp: 1234 is a number
temp ‘1234’ is a string
temp: ‘null’ is string
temp: null is undefined

print temp from above will do 1234, or 1234, or null, or undefined