paula updates to ajs #331
Open
paula updates to ajs #331
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Updated so that: - some chunk rearrangement and simplification to be more straight forward - accordingly renamed some chunks (e.g. prep_nonDHIS_data not appropriate as there are several chunks) - Removal of summarytools as doesn't work - Clearer explanations - Age calculations corrected (note still need to check 2 months = 62 days or 56? Show years at 24 months onwards?). - Other typos corrected Note it's now aligned with the measles outbreak recode Rmd.
aspina7
commented
Mar 20, 2025
|
|
||
|
|
||
| #linelist_raw <- rio::import(here::here("Data", "linelist.xlsx"), which = "Sheet1") | ||
| linelist_raw <- import("dat_msl.RDS") |
There was a problem hiding this comment.
think you just forgot to remove your line of rds data?
| This chunk demonstrates how to recode factor variables. You will need to add to | ||
| it based on your dataset by comparing to the variables in the standard data | ||
| dictionary. | ||
| This chunk is for recoding factor (categorical) variables. You will need to edit this section to recode the values in your dataset to suit the values in the expected linelist format. You can look at the data dictionary object (`recode_dict`) and the outputs from the `browse_data` chunk to write the correct code. |
There was a problem hiding this comment.
can stick to 80 characters a line pls? theres a setting in rstudio to have guideline show
| age_num >=2 & age_unit == "Year" ~ age_num, | ||
| age_num >= 24 & age_unit == "Month" ~ as.numeric(age_num * 12), | ||
| age_num >=2 & age_unit == "Year" ~ as.numeric(age_num), | ||
| age_num >= 24 & age_unit == "Month" ~ as.numeric(age_num / 12), |
| age_num >=2 & age_unit == "Month" ~ age_num, | ||
| age_num >=56 & age_unit == "Day" ~ as.numeric(age_num / 30), | ||
| age_num >=2 & age_num <24 & age_unit == "Month" ~ age_num, | ||
| age_num >=62 & age_num < 730 & age_unit == "Day" ~ as.numeric(age_num / 30), |
There was a problem hiding this comment.
happy with leaving as 62, the reason i put over 56 is that if they are born jan or feb then they could be two months old at 59 days .... with the added brainfart that i just multiplied 28 by 2.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated so that:
Note it's now aligned with the measles outbreak recode Rmd.