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

Include ICD11 mappings #486

Merged
merged 2 commits into from
Apr 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
104 changes: 84 additions & 20 deletions src/mappings/doid.sssom.tsv

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/mappings/gard.sssom.tsv
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
# skos: http://www.w3.org/2004/02/skos/core#
# sssom: https://w3id.org/sssom/
# license: https://w3id.org/sssom/license/unspecified
# mapping_set_id: https://w3id.org/sssom/mappings/fba76c0f-e243-4b76-84be-13031a56bcfa
# mapping_set_id: https://w3id.org/sssom/mappings/a2d27ac8-e35c-4370-a09d-803978eaa02f
subject_id subject_label predicate_id object_id mapping_justification
obo:GARD_1 GRACILE syndrome skos:exactMatch Orphanet:53693 semapv:UnspecifiedMatching
obo:GARD_1 GRACILE syndrome skos:narrowMatch OMIM:603358 semapv:UnspecifiedMatching
Expand Down
2 changes: 1 addition & 1 deletion src/mappings/icd10cm.sssom.tsv
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@
# skos: http://www.w3.org/2004/02/skos/core#
# sssom: https://w3id.org/sssom/
# license: https://w3id.org/sssom/license/unspecified
# mapping_set_id: https://w3id.org/sssom/mappings/505bfd44-9435-45eb-be59-3c19aa83d3e3
# mapping_set_id: https://w3id.org/sssom/mappings/c15beb32-c0a5-4a9f-bff3-337c52503878


2 changes: 1 addition & 1 deletion src/mappings/icd10who.sssom.tsv
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@
# skos: http://www.w3.org/2004/02/skos/core#
# sssom: https://w3id.org/sssom/
# license: https://w3id.org/sssom/license/unspecified
# mapping_set_id: https://w3id.org/sssom/mappings/69a61798-1eb3-45d6-a3fa-bcd03dc5cf25
# mapping_set_id: https://w3id.org/sssom/mappings/8e08c2db-8891-4a5e-990e-1a50b1b83041


2 changes: 1 addition & 1 deletion src/mappings/ncit.sssom.tsv
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
# skos: http://www.w3.org/2004/02/skos/core#
# sssom: https://w3id.org/sssom/
# license: https://w3id.org/sssom/license/unspecified
# mapping_set_id: https://w3id.org/sssom/mappings/9367551c-034c-40d2-881a-2a73c73020a2
# mapping_set_id: https://w3id.org/sssom/mappings/644cb230-28fb-4860-b973-21bd6950b861
subject_id subject_label predicate_id object_id mapping_justification
NCIT:C103222 Popliteal Fossa oboInOwl:hasDbXref UBERON:0013069 semapv:UnspecifiedMatching
NCIT:C103384 Common Iliac Lymph Node oboInOwl:hasDbXref UBERON:0015878 semapv:UnspecifiedMatching
Expand Down
5,028 changes: 2,541 additions & 2,487 deletions src/mappings/omim.sssom.tsv

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/sparql/rm_xref_by_prefix.ru
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ WHERE {
oboInOwl:hasDbXref ?xref ;
?p1 ?o2 .
}
FILTER( STRSTARTS(str(?xref), "UMLS_ICD9CM_2005_AUI:") || STRSTARTS(str(?xref), "ICD11:") || STRSTARTS(str(?xref), "SNOMEDCT_US_") || STRSTARTS(str(?xref), "IMDRF:") || STRSTARTS(str(?xref), "url:") )
Copy link
Contributor

Choose a reason for hiding this comment

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

Just commenting on my understanding.
I see what's going on here to address #140. So this deletion query gets run on the DOID and NCIT components, removing these xrefs. You're removing ICD11 from this query, so thus we're adding the mappings back from these files, as they won't be removed any longer.

I thought that #140 had something to do with my new ICD11Foundation work but it does not.

FILTER( STRSTARTS(str(?xref), "UMLS_ICD9CM_2005_AUI:") || STRSTARTS(str(?xref), "SNOMEDCT_US_") || STRSTARTS(str(?xref), "IMDRF:") || STRSTARTS(str(?xref), "url:") )
Copy link
Contributor

Choose a reason for hiding this comment

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

Merging into develop or main

@hrshdhgd I see that this is merging into main, but basically from now on, per #405 (which isn't completed yet) we're going to follow the normal git flow of main <-- develop <-- feature branches.

We could change the base on this PR but IDK how much it matters. I update develop from master regularly as needed anyway.

Copy link
Contributor

Choose a reason for hiding this comment

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

@hrshdhgd I approve as well as Nico. Prolly good to merge. I can update develop later.

}