Too many API calls in default weather module after update

[Perilax]

Hello together,

Since the April update, I have an issue where the default weather module sends too many requests to the OpenWeathermap API. I reach the free limit of 1,000 requests after just a few hours.

Here’s an extract from my config.js:

 {
                        module: "weather",
                        position: "top_right",
                        config: {
                                weatherProvider: "openweathermap",
                                apiVersion: "3.0",
                                weatherEndpoint: "/onecall",
                                lat: "XXXXXX",
                                lon: "XXXXXX",
                                location: "XXXXXX",
                                showSun: true,
                                type: "current",
                                appendLocationNameToHeader: false,
                                apiKey: "XXXXXXX",
                                updateInterval: 10*60*1000, // Update every 10 minutes
                        }
                },
                {
                        module: "weather",
                        position: "top_right",
                        header: "Wetter Vorschau",
                        config: {
                                weatherProvider: "openweathermap",
                                type: "forecast",
                                location: "XXXXXX",
                                apiVersion: "3.0",
                                weatherEndpoint: "/onecall",
                                lat: "XXXXXX",
                                lon: "XXXXX",
                                appendLocationNameToHeader: false,
                                apiKey: "XXXXX",
                                updateInterval: 10*60*1000, // Update every 10 minutes
                                fade: false,
                                roundTemp: true,
                                ignoreToday: true,
                        }
                },

Has anyone a solution to solve this problem ?

[sdetweil]

@Perilax @kristjanesperanto will have to answer, there was a big rewrite of the data collector side of weather.

[stephanvdplas]

@Perilax

I think you need these parameters:

			updateInterval: 60 * 60 * 1000,
			retryDelay: 60 * 60 * 1000,          // retry only once per hour

[Perilax]

@stephanvdplas

Is „retryDelay“ really working ? I can‘t find it in the documentation.

I have two modules where I request the API every 10 minutes. So it should be around 300 requests per day.
By far less than 1000.

[KristjanESPERANTO]

Sam is right, that was caused by the big rewrite (which resolved other issues, particularly a fundamental architectural one).

A fix is already in the develop branch: https://github.com/MagicMirrorOrg/MagicMirror/pull/4092

Are you familiar enough with Git to test the develop branch?

[evroom]

@perilax
Or just a quick work-around as the default for updateInterval for openweathermap is 10 minutes anyway.
Try to comment out those 2 lines with //.

@kristjanesperanto
Quick response; quick fix 👍

[Perilax]

@KristjanESPERANTO

I’ll test the develop branch tomorrow.
Thanks for the quick reply.

@evroom I added updateInterval to my config.js only after the problem occurred, because I thought it might be causing the issue.

[sdetweil]

@Perilax see this link for how to get the develop branch
https://forum.magicmirror.builders/post/86422