lcpz / lain

Awesome WM complements
GNU General Public License v2.0
1.06k stars 210 forks source link

Weather forecast widget, again. #499

Closed cyberpunkrocker-zero closed 3 years ago

cyberpunkrocker-zero commented 3 years ago

Since Openweathermap changed its API licenses and forced us to use '5 day forecast' instead of the '16 day forecast', the values of "temp_min" and "temp_max" has became meaningless, as explained in https://openweathermap.org/forecast5

Please, do not confuse min/max parameters in our weather APIs.

In 5 day / 3 hour forecast API, Hourly forecast API and Current weather API - temp_min and temp_max are optional parameters mean min / max temperature in the city at the current moment just for your reference. For large cities and megalopolises geographically expanded it might be applicable. In most cases both temp_min and temp_max parameters have the same volume as 'temp'. Please, use temp_min and temp_max parameters in current weather API optionally. In 16 Day forecast - min and max mean maximum and minimum temperature in the day.

"temp_min" and "temp_max" both show always the same value, which is equal to "temp". They are not real min/max temperatures any more. For the current Openweathermap situation the forecast notification popup should be modified so that it shows only the value of "temp"