Skip to content
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

Mappings for patient, family history, social history and location #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

janhaviel
Copy link
Collaborator

Added mappings for the patient, family history, social history and location tables from the ACEP data dictionary, corresponding to tickets DAN-55, DAN-56, DAN-57, DAN-58

Copy link
Collaborator

@pverma12394 pverma12394 Oct 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • social_history_observed_value and family_history_observed_value will map to obersvation_concept_id alongwith observation_source_value. Reason: a source value is mapped to a concept ID, whereas source codes are mapped to source_concept_ids. "A source code may not always be a concept ID" , therefore all source codes/concept IDs are mapped to source_concept_id attributes

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Type codes in the acep will not map to obersvation_concept_id for the same reason as above. They are mapped to observation_source_concept_id only

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense thank you! We got the rationale behind using the observed_value to derive concept_id but we were wondering if we can implement a logic here that if the social_history_type_code or family_history_type_code does not have a corresponding concept_code in the concept table then we should derive the observation_concept_id from the social_history_observed_value and family_history_observed_value? Or would that cause any issues?

Copy link
Collaborator

@pverma12394 pverma12394 Oct 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the code is available and the choice of standard vocabulary is same as source, then we use the code to fetch the concept ID, else if code is unavailable or missing then we use the value to map to the corresponding concept ID. That logic is part of the vocabulary mapping process which will be discussed later.

@ishitasati11 FYI

Janhavi Mada added 2 commits October 18, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants