feat: ✨ convert REDCap dictionary to resource properties#35

Merged

lwjohnst86 merged 13 commits intomainfrom

feat/convert-to-resources

Apr 7, 2026

Collaborator

martonvago commented Mar 20, 2026 •

edited

Loading

Description

This PR adds the ability to convert a saved REDCap data dict to resource properties.
Followed this plan: #23 (comment)
Does not include a resource for events/visits. Not sure if we want that?

Closes #25 closes #26

This PR needs an in-depth review.

Checklist

Formatted Markdown
Ran just run-all


          feat: ✨ convert redcap data dict to resource properties

92fb93a

martonvago self-assigned this

martonvago added this to Data development

github-project-automation bot moved this to Todo in Data development

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_data_dict_to_resource_properties.py Outdated Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved


          fix: 🐛 fix description whitespace

c3d3ac8

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved


          refactor: ♻️ use optional

096ce2c

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago moved this from Todo to In Review in Data development

martonvago requested review from K-Beicher and lwjohnst86

March 20, 2026 11:52

martonvago marked this pull request as ready for review

March 20, 2026 11:56


          fix: 🐛 access FieldType properly after Sprout update

9c9f995

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved


          feat: ✨ use text_validation_type_or_show_slider_number to pick a bett…

…er field type

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

lwjohnst86 requested changes

View reviewed changes

Member

lwjohnst86 left a comment

Nice start! It would be better to actually see what the properties output is though.

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_data_dict_to_resource_properties.py Outdated Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_data_dict_to_resource_properties.py Outdated Show resolved Hide resolved

scripts/redcap_data_dict_to_resource_properties.py Outdated Show resolved Hide resolved

scripts/redcap_data_dict_to_resource_properties.py Outdated Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

github-project-automation bot moved this from In Review to In Progress in Data development

martonvago added 2 commits

March 25, 2026 11:45


          refactor: ♻️ review markups

b254c2f


          feat: ✨ extract min and max constraints

f930e85

martonvago and others added 5 commits

March 25, 2026 13:30


          fix: 🐛 rename visit to event everywhere

2f9d80a


          feat: ✨ add generated resource properties in package properties


          fix: 🐛 add back Danish cpr mask

1c4bb24


          feat: ✨ add center field

a5a6e40


          feat: ✨ generate datapackage.json from CPH metadata

1d1542f

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

martonvago commented

View reviewed changes

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

Collaborator Author

martonvago commented Mar 25, 2026

I added the generated resource properties to the package properties and updated main.py. Then I ran main.py on GenomeDK and committed the generated datapackage.json.

martonvago moved this from In Progress to In Review in Data development

martonvago requested a review from lwjohnst86

March 25, 2026 14:04


          Merge branch 'main' of https://github.com/onlimit-study/feasibility-data

b97f2e5

 into feat/convert-to-resources

lwjohnst86 approved these changes

View reviewed changes

Member

lwjohnst86 left a comment

Nice 🎉

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

scripts/redcap_dict_to_properties.py Show resolved Hide resolved

github-project-automation bot moved this from In Review to In Progress in Data development

lwjohnst86 changed the title ~~feat: ✨ convert redcap data dict to resource properties~~ feat: ✨ convert REDCap dictionary to resource properties

lwjohnst86 merged commit 8a0bc24 into main

lwjohnst86 deleted the feat/convert-to-resources branch

April 7, 2026 07:40

github-project-automation bot moved this from In Progress to Done in Data development

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet