Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/Makefile Automation #7

Merged
merged 4 commits into from
Mar 16, 2020

Conversation

desholmes
Copy link
Contributor

@desholmes desholmes commented Mar 15, 2020

Overview

Introduce make commands to automate the process of gathering the daily UK data. Updated README.md.

Changes

  1. Introduced Makefile to automate the England daily tasks
  2. Updated tools/extract_totals.py to append data to ./data/covid-19-totals-uk.csv if the downloaded temp HTML file contains today's date
  3. Updated .gitignore to exclude vscode and temp HTML files
  4. Updated README.md with usage instructions for the make commands

Steps to test

  1. Check out this branch
  2. Make commands: Follow the instructions in the Make commands section of README.md
  3. extract_totals.py: Updated usage instructions in Manually running scripts to include a second arg for ./data/covid-19-totals-uk.csv

@desholmes
Copy link
Contributor Author

@tomwhite Added a Makefile to semi-automate the England tasks.

@tomwhite
Copy link
Owner

@desholmes this is amazing! I will give it a run through tomorrow. Many thanks.

@tomwhite
Copy link
Owner

+1. I've run some light testing on this and it looks great. Especially glad you found a stable URL for the CSV on ArcGIS.

It would be nice if we could use Make in such a way that it doesn't repeatedly download a file (e.g. make england-totals-download will always re-download even if the file is there locally), but that can be improved later.

@tomwhite tomwhite merged commit e577602 into tomwhite:master Mar 16, 2020
@desholmes desholmes deleted the feature/makefile-automation branch March 16, 2020 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants