csparpa / pyowm

A Python wrapper around the OpenWeatherMap web API
https://pyowm.readthedocs.io
MIT License
789 stars 171 forks source link

Added size argument weatherapi25.weather.weather_icon_url #337

Closed Wesley-Vos closed 3 years ago

Wesley-Vos commented 3 years ago

OWM released a new set of icons which are looking better and have different sizes. The new URL was already replaced, the size is now added as an argument in the weather_icon_url. If no argument is given, the default size (same size as the old icons) is returned. Other options are 2x and 4x. The changes are discussed in #324.

csparpa commented 3 years ago

@Wesley-Vos thanks for that! It looks like the Travis-CI build failed... because of a bad string formatting in a unit test - can you take a look please ? This is the relevant info from Travis: https://travis-ci.org/github/csparpa/pyowm/jobs/727686501#L1098

Also, don't forget to add yourself to the CONTRIBUTORS.md file! For the records!