BCCN-Prog / weather_2016

For the BCCN 2016 advanced programming project
3 stars 1 forks source link

Add prediction_time to scraping dictionary #51

Closed clauslang closed 8 years ago

clauslang commented 8 years ago

From #23:

Please drop a quick line here once this is done for your script so that we have an overview.

clauslang commented 8 years ago

Done for owm.

akresnia commented 8 years ago

done for timeanddate

janfb commented 8 years ago

done for wetter.com

gkBCCN commented 8 years ago

My commit closed the issue on GitHub accidentally. My commit only resolves this issue for wetter.de.

denisalevi commented 8 years ago

Just to clarify: We now have multiple files for the same date and city but e.g. downloaded with 12h difference. So I have to handle it myself, that the predictions e.g. at 5am and 5pm end up in different dictionaries that will then individually we loaded into the database (with the only difference to distinguish them being the 'prediction_time' value)?

clauslang commented 8 years ago

Yes. We might in the future decide that only one prediction per day is loaded into the database. I think the reasoning for downloading twice a day was mainly to be safe in case one of the downloads fails. (Even though it is possible that there is an interesting analysis case comparing predictions of the same day at different times for a subset of the providers.)

clauslang commented 8 years ago

@inesw finished it for wunderground

denisalevi commented 8 years ago

I do have the prediction time in my dictionaries, but currently data saved at different hours of the same day (e.g. 5am and 5 pm) end up in the same dictionary. I need to fix that, will do that later today.

denisalevi commented 8 years ago

done for accuweather