-
Notifications
You must be signed in to change notification settings - Fork 10
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
TIMS/N3C: [OMOP <--> ICD10CM, etc] --> SSSOM --> FHIR ConceptMap #140
Comments
@Sigfried This isn't related to TermHub but Stephanie doesn't have any code for this yet, and it uses @stephanieshong FYI, I created this issue for Nico to respond to. @matentzn Here you are! |
|
@matentzn - ok. Thank you. I will give you some example of one to many map that you can comment on. |
@matentzn Awesome answers, thanks. I spoke with @stephanieshong and got her what she needs. She was asking about how to construct the
|
@matentzn I added a new question (4): What CURIE to use for OMOP? For now I randomly picked https://www.ohdsi.org/data-standardization/the-common-data-model/ , but I know that this is wrong. I don't know if there is any URL pattern for pulling up pages that show information about OMOP codes. |
In all my projects we use All codes resolve there. |
Thank you. |
@matentzn - thank you I will use athena link to resolve in the curie map. I am just making a note for myself here:
|
@Sigfried Just FYI TIMS woulda been the better place to put this issue, but w/ the discussion here so far, too late to move it. I'll make a mirror issue and link it. |
@joeflack4 - did you remove the sssom_map branch? |
@joeflack4 Stephanie is asking me where the sssom_mapping branch and files went. I didn't know. I had her add them to the develop branch: 077da0c |
@joeflack4 - checked in my sample files under the sample_mapping_files directory |
Hey @stephanieshong, I know we talked about this on Teams. Yeah, I'm not sure; I don't remember us creating that branch. I think @Sigfried and I have not yet decided where mapping content should go. Maybe we need a new For now I agree they are fine there. Could always just add these to a standalone repo as well. |
Yes, I created the branch from my machine. But I think I created from the sub-directory and then later removed the .git so i think it got lost. |
@joeflack4 - I will create a branch from the main and use that branch for the SSSOM mapping files. |
A ---[narrowMatch]-->B means that B is narrower than A. |
Overview
Stephanie is doing some mapping work, I think for N3C (I'm not sure the exact nature of it). There is a desire create SSSOM for these mappings. After creating the SSSOM files, we'll convert to FHIR ConceptMap and load onto the TIMS server.
Tasks
Questions
mapping_justification
: What is the valid range? Is there an enumeration of permissible values somewhere?mapping_justification
: We know that OMOP has mapped to various terminologies using automation. However, we don't know how it was done. Doessemapv:lexicalMatch
have an abstract parent classsemapv:match
that we could use? Or is there something insemapv
(or elsewhere) for 'unspecified algorithm'?mapping_justification
, if we don't know the details for "use mapping_tool, mapping_tool_version, confidence , match_string, subject_preprocessing, object_preprocessing,", should we leave these fields out? Or should we include them and leave them blank or populate them with some 'unknown' value or null value?Additional information
Slack Conversation
Related
timsbiomed/issues#59
#145
The text was updated successfully, but these errors were encountered: