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

Study definition preprocessing #102

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

harrytmason
Copy link
Contributor

@harrytmason harrytmason commented Feb 17, 2025

This branch is hopefully just the previous branch (study_definition) with only 4 different files

create_project_actions - now with the extra 3 actions, and an option for including the describe*.txt files
project.yaml - following on from above
preprocess_data.R - Updated to allow the inclusion of describe*.txt files, and updates to keep in line with this branch
modify_dummy_data.R - minor updates and formatting

The version of project.yaml with preprocessing function calls
Allowing preprocess_data_* functions to have an additional arguement "describe_print" or "no_describe_print", which dictates whether the describe*.txt files are produced (in this version, they are not)
Some slight formatting changes for readability, and also some edits to ensure the Moderna vaccine is being referenced correctly
Adding a flag to describe_print, to decide whether or not to print out the describe.txt files.

Additionally, the "action" variable within the "action" function was renamed to make it clearer that the function is not using recursion.

Also, some small formatting changes for readability
Had some junk code come in from working on multiple branches, should be cleaned up now
Not fully up to date with study_definition branch, but a version that works and has fiarly consistent names
Recently committed study_definition to main, this brings the codelists in line with that updated example
Updating the pre-preprocessing part of the pipeline that was created on the study_definition branch
@harrytmason harrytmason requested a review from ZoeMZou February 17, 2025 23:26
Updating the outputs of preprocessing to output/describe or output/datset_clean (and some minor formatting improvements)
Changing metadates.R -> study_dates.R
Adding output subdirectories
Some other naming updates to the project.yaml
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