-
Notifications
You must be signed in to change notification settings - Fork 1
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
📝 Condition and Family Module Drafts #54
Conversation
🚀 IG Site Preview DeployedLatest commit: f790be8 |
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.
Some small comments
|
||
|
||
|
||
This distinction is similar to those in OMOP and FHIR, though it does not map precisely. Critically in OMOP and FHIR, Condition_occurrence and Condition are ALWAYS a positive assertion (though they may be wrong). “Assertions of history of disease” are observations in OMOP, eg, ICD9CM V-codes like “Personal history of malignant neoplasm of breast” go in Observation. |
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.
I'm wondering if calling it "Condition Module" is going to make people assume we're using the FHIR Condition resource under the hood? And that this piece is too buried in the versus being a bit up front that it's actually an Observation and/or that we're talking more generically about it versus the specific Condition resource?
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.
One other thought on this: The "Condition Module" should ALSO have a condition profile IMO. The "Condition Summary" resource would help round this out. I think we want people looking for "Conditions" or looking for "Condition Resources" to come here to understand what we are doing.
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.
I am going to merge the current state of these changes but open a ticket to add updates @RobertJCarroll is describing here.
Motivation
We need module documentation for family and condition
Approach
Starts a draft for each of these pages