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

Fixed a couple data issues preventing code from the book from running as-is #66

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

Conversation

dsconnell
Copy link

Inconsistencies between the data on github and the code in the book prevented some joins and data cleaning operations. Please see my commit descriptions for details!

…lumn

Following the code in the book, you cannot "join the deaths and population data based on common columns" https://epirhandbook.com/en/standardised-rates.html#clean-populations-and-deaths when there is no Country column in the demographic data.
The code in the "Clean reference population" section https://epirhandbook.com/en/standardised-rates.html#load-reference-population expects the age category column to be called "age_cat5" not "AgeGroup".
@netlify
Copy link

netlify bot commented Nov 14, 2021

✔️ Deploy Preview for epirhandbook ready!

🔨 Explore the source changes: 1a94f7e

🔍 Inspect the deploy log: https://app.netlify.com/sites/epirhandbook/deploys/6193ff42c596cb0007406184

😎 Browse the preview: https://deploy-preview-66--epirhandbook.netlify.app

this introduced NAs in the left_join in the "Clean populations and deaths" section https://epirhandbook.com/en/standardised-rates.html#clean-populations-and-deaths
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.

1 participant