-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make checkbox fields explicit #199
Conversation
Summary of the missing optional fields for the ccp-ghana parser:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
@sadiekelly sorry to bug again... I thought this covered all the bases for checkbox fields by looking for fields with ___x in the name, but having looked again at some of the data files it's not clear to me that a lot of the symptom data wasn't also recorded as checkbox-like - e.g., the Guinea, Ghana & Uganda-v2 (these are just the first ones I checked) seem to have a data structure that suggests they use 0 as a default for admission symptoms/comorbidities, even though there are 'unknown' as options. Any insights? |
hi @pipliggins for those databases you've mentioned, it is fine to treat these not as checkboxes - they were either radio button or dropdown lists for the Yes/No/Unknown options meaning that one option should be selected and if none were selected then the cell in the csv for that data item would just be blank, there would not be a default entry for these. |
Hi @sadiekelly if you have the spare capacity that would be great! The well filled-in radio fields make spotting these easily a bit more difficult, good to know. |
hi @pipliggins I've checked through the data dictionaries, most are REDCap format so will be covered by the ___x. Of the couple that are not, generally checkboxes were not used. I think predicovid may have been one, and the checkbox fields there are not mapped so no changes needed. Otherwise looks like the checkboxes have been accounted for so all good! |
Fab, thanks for checking this @sadiekelly! |
No description provided.