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.

Sign up for GitHub

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

Jump to bottom

R data preparation to add age rescaling and determinism #66

Merged
merged 61 commits into from
Feb 14, 2024
Merged

R data preparation to add age rescaling and determinism #66

merged 61 commits into from
Feb 14, 2024

Conversation

sgreenbury
Copy link
Collaborator

Closes #63.

sgreenbury added 30 commits July 27, 2023 16:38
@sgreenbury
Update renv
6aa7295
@sgreenbury
Add run 1 versions of scripts
d0e583e
@sgreenbury
Remove code not required for run 1
a526566
@sgreenbury
Add separate source for run 1 functions
e2a0022
@sgreenbury
Make separate scripts for SPC function runs
ced21f1
@sgreenbury
Revise functions for run 1
ba1c5d5
@sgreenbury
Add separate SPC_loadWorkspace scripts
f96168e
@sgreenbury
Rename scripts for run 1 and 2
3a8dfbf
@sgreenbury
Fix missing outpath
1216d76
@sgreenbury
Update renv lock
4e03ad6
@sgreenbury
Update to script for step 3 (#63)
9334b6d
@sgreenbury
Make deterministic from single seed
4dab1d3
@sgreenbury
Remove unused code, linting and formatting
efa8d13
@sgreenbury
Formatting and linting for functions
d59ee7b
@sgreenbury
Formatting, add cores arg
1501cc0
@sgreenbury
Move run1 to subdirectory
526e662
@sgreenbury
Initial age_rescaling script
21a9dfc
@sgreenbury
Adjust age_rescaling script for LADs
0a9d597
@sgreenbury
Remove ethnicity from output as unused
4027822
@sgreenbury
Rounding for income
ee2649a
@sgreenbury
Rename path, add script
af52889
@sgreenbury
Initial completed refactor
66a79cc
@sgreenbury
Linting refactor
6051a8d
@sgreenbury
Fix coding
1d3d087
@sgreenbury
Rename function, add print
74877d2
@sgreenbury
Revert renames for enrichment after rescaling
5181ecc
@sgreenbury
Move age_rescaling script
5b64a44
@sgreenbury
Remove unused scripts
b693aee
@sgreenbury
Update rescaling script
3ab6474
@sgreenbury
Refactoring and adding comments
8b0edc7
@sgreenbury sgreenbury requested a review from HSalat January 19, 2024 11:30
@sgreenbury
Copy link
Collaborator Author

sgreenbury commented Jan 19, 2024 •
edited
Loading

Upon re-running steps 1 (all outputs in Data/prepData) and step 2 (tested for E06000001 and E09000001), the outputs are now deterministically produced from the given random seed on a given machine.

HSalat
HSalat approved these changes Feb 9, 2024
View reviewed changes
Copy link
Collaborator

@HSalat HSalat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite interesting! I should learn how to do all this.

@sgreenbury
Copy link
Collaborator Author

Quite interesting! I should learn how to do all this.

Thanks for taking a look and the review!

@sgreenbury sgreenbury merged commit 476ad88 into main Feb 14, 2024
2 checks passed
@sgreenbury sgreenbury deleted the 63-r-data-prep branch February 14, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HSalat HSalat HSalat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make first step (raw_to_prepared.R) fully runnable
2 participants
@sgreenbury @HSalat