generated from opensafely/research-template
-
Notifications
You must be signed in to change notification settings - Fork 1
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
harrytmason
wants to merge
10
commits into
main
Choose a base branch
from
study_definition_preprocessing
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
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
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
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.
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