Skip to content
This repository has been archived by the owner on Mar 7, 2021. It is now read-only.

Adds data for India #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sakethramanujam
Copy link

Adds data-sets for India, sourced from covid19india.org #1

- name: Convert IT CSV to sqlite db
run: csvs-to-sqlite it/data/national-data.csv it/data/it-national.db
- name: Convert UK cases CSV to sqlite db
run: csvs-to-sqlite uk/data/uk-national-cases.csv uk/data/uk-national-cases.db
- name: Convert UK deaths CSV to sqlite db
run: csvs-to-sqlite uk/data/uk-national-deaths.csv uk/data/uk-national-deaths.db
- name: Convert IN cases time series to sqlite db
run: csvs-to-sqlite in/data/cases_time_series.csv in/data/in-cases-timeseries.db
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we have quite a few data sets here for IN, I would suggest we conver them into a single sqlite DB, this can be done by passing in all the csv's followed by the DB name which I would call something in in/data/in-national-data.db or something. This will create a table per CSV. See here for an example in the docs - https://github.com/simonw/csvs-to-sqlite

I'll go back and do this for UK soon.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will resolve by weekend and push

@@ -0,0 +1,89 @@
dailyconfirmed,dailydeceased,dailyrecovered,date,totalconfirmed,totaldeceased,totalrecovered
1,0,0,30 January ,1,0,0
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the format the dates come in from the API? It's a bit random and may not serialise well into a date field :/ maybe we could convert this into a more standard date format and just set the time to 00:00?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's doable. will implement

@elliotblackburn
Copy link
Owner

Thanks so much for this by the way! I wrote out these comments but forgot to submit them :( I've done it now! So sorry for the delay

@sakethramanujam
Copy link
Author

that's okay. it happens. Will push the updates asap.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants