docs: ✨ Initial draft of functions to extract osdc population #71
Conversation
signekb
changed the title
|
I revised the doc and images a bit, mostly adding the input register source, converting it to a flow chart to show the interconnection with some of the functions and output data as input to other functions. I also added a function convention section so we have at least a guidelines for naming things. |
| - `calculate_pregnancy_index_date_for_mc_visits_wo_end_date()` (this | ||
| might be removed with the inclusion of the birth register) |
There was a problem hiding this comment.
I don't really know what this does.
vignettes/images/function-flow.png
Outdated
There was a problem hiding this comment.
@Aastedet you'll have to confirm if the input registers are correct for each inclusion/exclusion function.
There was a problem hiding this comment.
Very niceeee @lwjohnst86. I really like the clarity of the naming scheme 🎉
Co-authored-by: Signe Kirk Brødbæk <[email protected]>
|
@lwjohnst86 @Aastedet I have updated this again based on the feedback I got from Luke, so I'll re-request reviews :) |
Co-authored-by: Luke W. Johnston <[email protected]>
Co-authored-by: Luke W. Johnston <[email protected]>
Description
This PR describes the functions for extracting a diabetes population using the
osdcpackage.See issue #67.
As we have discussed, this is an initial draft meant to help us specify exactly what we want to build.
So please chime in with questions, suggestions, and corrections.
As @Aastedet wrote here there is an overlap between #69 and this PR. @lwjohnst86 can you help us in terms of what should be in each post?