Version 2.30.0 weather module, stuck loading, worked fine before upgrade

sdetweil

@Rberry91 v3 does not return the right location,
location/id are not used in v3, only lat/lon

use the module header property

module:…
header:“…”

the set the weather config option

appendLocationToHeader:false,

Rberry91

Now I have an message to update the MMM-GooglePhotos module. How do I do that without causing problems like before. I did try your code and it didn’t seem to work but I am most likely doing something incorrect

sdetweil

@Rberry91 didnt try my code? i do not supply anything that will update modules.

my upgrade script is only for base MagicMirror

every module upgrade is the same

cd ~/MagicMirror/modules/modulename
git pull
xxx see the module doc on what to do after
sometimes you need an npm install, sometimes not

Rberry91

@sdetweil

Hi again,
The MMM- GoohlePhotos was not updating so I re installed and am know getting a error of missing creditionals please help

sdetweil

@Rberry91 now you have to follow the module instructions to get authorization again

Rberry91

@sdetweil
I reverted to the older version and I have tried updating using your command and other ones and still doesn’t update tell me that “pulling without specifying how to reconcile divergent branches is etc”

The other problem I am having is the weather, the forecast works but the current Weather doesn’t and they are the same code.

sdetweil

@Rberry91 divergent branches sounds like you have both modules trying to merge to the same folder

delete then git clone again
the nan says the apikey is wrong

and you have a typo, MagicMirror is case sensitive

butchkemper

The configuration parameters I use are:

module "MMM-OpenWeatherForecase"
latitude and NOT lat
longitude and NOT lon
apikey and NOT apiKey   <----  Notice the "K"

Some MM configuration keywords have abbreviations but not all. The module README file gives the values for the keywords.

Also, since your apikey has been revealed to the world, you should cancel the posted key and generate a new key. Always keep the apikey secret.

Butch

butchkemper

@sdetweil I noticed this discretion between the Weather module documentation:

The “Weather” documentation has “apiKey” - large case “K”,
The “MMM-OpenWeatherForecast” documentation has “apikey” - small case “k”.

Butch

sdetweil

@butchkemper modules can go anything they want
what is used internally can be anything required

I looked at the doc for the default weather module before posting above. as there are two instances, one works the other doesn’t. apikey is the issue

Assassins

Work on 2.29.0 with api2.5

But now on 2.33.0
It say “NaN”
I get the error 401, it say the apiKey i wrong, but is write from my api key from the website.
How can i get a apiKey to api3.0 without to paid for it. ???
The api2.5 still work, but not on this modul.

    {
      module: 'weather',
      position: 'bottom_left',
      header: 'Ballerup',
      config: {
         lat: "55.7317",
         lon: "12.3633",
         apiKey: "xxxxx765xxxx98xxxjhx888"
       }
    },

sdetweil

@Assassins you cannot get V3 apikey from open weather without providing a credit card.
they don’t charge unless you go over 1000/request per day.

now, you CAN make 2.5 work (if they still accept api requests they keep telling us they are going to do (have done!!)) just be warned that they can/will end this function sometime…

in the module config section
apiKey: 2.5_key
apiVersion: 2.5

and

weatherEndpoint: “current” OR “forecast”

edited

Assassins

@sdetweil
I can only see the header now, where is write Ballerup

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: "2.5",
    weatherEndpoint: "/current",
     lat: "55.7317",
     lon: "12.3633",
     apiKey: "xxxxx765xxxx98xxxjhx888"
   }
},

sdetweil

@Assassins

sorry

weatherEndpoint: “current” or 'forecast" , no /

also, the old api use location or locationID, not lat/lon
and I see you can’t look them up anymore on OWM site

here is from the old doc

locationID	Location ID from OpenWeatherMap This will override anything you put in location.

Leave blank if you want to use location. <------=======

Example: 1234567
Default value: false

Note: When the location and locationID are both not set, the location will be based on the information provided by the calendar module. The first upcoming event with location data will be used.

======================================================================
location	The location used for weather information.

Example: 'Amsterdam,Netherlands'
Default value: false

Note: When the location and locationID are both not set, the location will be based on the information provided by the calendar module. The first upcoming event with location data will be used.

so it ‘seems’ that location by itself would do it.

Assassins

@sdetweil
This is the same, can only see the header.
Is work yesterday on MM 2.29.0

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: '2.5',
   apiBase: 'https://api.openweathermap.org/data/',
    weatherEndpoint: '/no',
    location: 'Ballerup',
    locationID: '2624341',
    apiKey: 'BBnjsdnsdfnsdj8789dsfjsdnfnXX',
    showHumidity: true,
    initialLoadDelay: 1500
  }
},

sdetweil

@Assassins said in Version 2.30.0 weather module, stuck loading, worked fine before upgrade:

apiBase: ‘https://api.openweathermap.org/data/’,
weatherEndpoint: ‘/no’,

no apiBase, that didn’t change

the weatherEndpoint is either “current” OR “forecast”

I originally gave them to you with leading / , so my comment was without the / (no /)

Assassins

@sdetweil
Is the same mate

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: '2.5',
    weatherEndpoint: 'current',    // i have try the forecast to
    location: 'Ballerup',
    locationID: '2624341',
    apiKey: 'xxjljkjsksldjkj3478bjjk',
    showHumidity: true,
    initialLoadDelay: 1500
  }
},

sdetweil

@Assassins seems it doesn’t work anymore… using my old 2.5 key I get code failure with openweathermap

but we changed the sample defaults to a free provider that might work

		{
			module: "weather",
			position: "top_right",
			config: {
				weatherProvider: "openmeteo",
				type: "current",
				lat: 40.776676,
				lon: -73.971321
			}
		},
		{
			module: "weather",
			position: "top_right",
			header: "Weather Forecast",
			config: {
				weatherProvider: "openmeteo",
				type: "forecast",
				lat: 40.776676,
				lon: -73.971321
			}
		}

Assassins

@sdetweil said in Version 2.30.0 weather module, stuck loading, worked fine before upgrade:

@Assassins seems it doesn’t work anymore… using my old 2.5 key I get code failure with openweathermap

Strange, works on my sd card with MM2.29.0 and also works on MMM-DynamicWeather and on MMM-WeatherOrNot with MM2.33.0.
Don’t know why openweathermap doesn’t work with MM2.33.0.
Besides that, it may have something to do with the code in MM2.33.0 that has been rewritten.