-
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
add dob_year and edit antimalarial #189
Conversation
Summary of the missing optional fields for the ccp-ghana parser:
|
Summary of the missing optional fields for the ccp-ghana parser:
|
Thanks @ekamau! I've made a couple of minor edits (mostly just adding excludeWhen = "none" where required), but this all looks good to me now. |
Left open until we decide how to deal with Washington (with age ranges rather than a single value). |
] | ||
|
||
[visit.treatment_antimalarial_type] | ||
combinedType = "set" |
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.
@ekamau I don't think other antimalarial treatments as indicated in antimal_cmtype, daily_antimal_cmtype and overall_antimal_cmtype have all been included - I couldn't see these in the parser in the main branch and the edits here only include chloroquine and hydroxychloroquine. So these additional ones could be mapped also (primaquine, quinine/mefloquine etc.)
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.
Thanks @sadiekelly - I have added the extra variables
{ field = "daily_exper_cmtype___2", values = { 1 = true } }, | ||
{ field = "exper_cmtype___1", values = { 1 = true } }, | ||
{ field = "exper_cmtype___2", values = { 1 = true } }, | ||
] |
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.
same comment as for RAPID re. the other antimalarials listed in *antimal_cmtype variables not yet mapped
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.
Thanks @sadiekelly - I have added the extra variables
Hi @pipliggins - kindly check the changes to experimental agent .. I moved or changed these to 'treatment_other'. I think we are also meant to move some experimental agent information to metadata. |
Summary of the missing optional fields for the ccp-cameroon parser:
|
Summary of the missing optional fields for the ccp-cameroon parser:
|
as in Miscellaneous bugs/inconsistencies #178