ManuelAlvarezC commented 4 years ago

Description

Add mobility data source Fixes #69

Checklist:

[x] I have read the CONTRIBUTING GUIDE and made sure this Pull Request is compliant with it.
[x] I have read the DATA MODEL and DATA SOURCES and made sure this Pull Request is compliant with it.
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ManuelAlvarezC commented 4 years ago

This finishes the work @DanielRobertNicoud started on #61. Which now will be closed.

wwymak commented 4 years ago

hmm, it looks like google actually now offers a csv download https://www.google.com/covid19/mobility/index.html (https://www.gstatic.com/covid19/mobility/Global_Mobility_Report.csv). Might be a bit more of a robust/accurate dataset to go with that rather than scraping pdfs? (sorry @ManuelAlvarezC I know you have spent quite a bit of time on the pdf parsers...)

(and you probably know the linter isn't too happy... :wink: )

ManuelAlvarezC commented 4 years ago

Hi @wwymak and thanks for the review!

I will start working in the changes right now!

And yes, it was a lot of work the parsing, but it's better this way.