Skip to content
This repository has been archived by the owner on May 8, 2020. It is now read-only.

Estimates data prep #8

Closed
wants to merge 2 commits into from
Closed

Estimates data prep #8

wants to merge 2 commits into from

Conversation

hnykda
Copy link
Contributor

@hnykda hnykda commented Mar 23, 2020 •
edited
Loading

@lagerros
Copy link

For John Hopkins, probably want to use "null" rather than "0" when we have no data

@gavento
Copy link
Contributor

gavento commented Mar 23, 2020

Hi! I am confused - why this contains config-local and some CSVs? Just to show them?

@hnykda
Copy link
Contributor Author

hnykda commented Mar 23, 2020 •
edited
Loading

Hi! I am confused - why this contains config-local and some CSVs? Just to show them?

Yes, just for that for reproducibility, it is definitely not something to be merged to master with data

@hnykda
Copy link
Contributor Author

hnykda commented Mar 24, 2020 •
edited
Loading

@gavento @janpipek @jez4 Jan P. just informed me that there was no notebook here, which makes this PR dumb 🤦‍♂️ - I didn't push it... Fixed

FYI I used pycountry.countries.search_fuzzy() quite successfully to get the iso codes where appropriate (that's how I created those maps).

@hnykda hnykda mentioned this pull request Mar 24, 2020
Closed
@hnykda hnykda added the invalid This doesn't seem right label Mar 24, 2020
This was referenced Mar 25, 2020
Closed
Closed
Closed
@hnykda hnykda closed this Mar 26, 2020
@hnykda hnykda deleted the estimates-case-data branch March 26, 2020 21:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
3 participants
@hnykda @lagerros @gavento