xofbd / wqu-weather-app

A demo web application for WorldQuant University Introduction to Data Science Module
GNU General Public License v3.0
7 stars 11 forks source link

Update layout and add a map #14

Closed Tim-Abwao closed 3 years ago

Tim-Abwao commented 3 years ago

This adds a map of the location using the folium python library. The app's style and requirements were also updated.

In addition, there seems to be a bug where the weather API returns an error message instead of the expected data. On some occasions when running locally, the weather API complained that the default 'user-agent', which it gave as the version-info of the requests package, was banned. So I added the header {'User-Agent': 'wqu_weather_app'} to the remaining GET requests.

Tim-Abwao commented 3 years ago

Hi, @xofbd!

I am doing well, thanks. I hope you are too.

xofbd commented 3 years ago

@Tim-Abwao Thank you for your contribution. Given my commitments, I think the best plan is for me to stop accepting PR for this repo. However, I encourage you to keep on developing your personal fork.