-
Notifications
You must be signed in to change notification settings - Fork 57
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
SCID CTPS1 to CID CTPS1 #8882
base: master
Are you sure you want to change the base?
SCID CTPS1 to CID CTPS1 #8882
Conversation
Closes #8178 - Renamed term to combined immunodeficiency due to CTPS1 deficiency, instead of a 'severe' combined immunodeficiency due to CTPS1 deficency - Added ORCID and PMID - Added previous label as a related synonym - Subclass of combined immunodeficiency and immunodeficiency. No long a subclass of severe combined immunodeficiency
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.
@yshwetar, please see questions and requested changes in lines.
In addition, if might be good to add the following:
- a logical definition (as long as it makes sense)
- a comment to say that this disease is not always severe (since other resources say it is). That would help users understand the differences.
src/ontology/mondo-edit.obo
Outdated
@@ -309897,7 +309897,7 @@ subset: ordo_malformation_syndrome {source="Orphanet:280576"} | |||
subset: orphanet_rare {source="Orphanet:280576"} | |||
subset: otar {source="MONDO:OTAR"} | |||
subset: rare | |||
synonym: "BANF1-related neurodevelopmental syndrome" EXACT CLINGEN_LABEL [https://search.clinicalgenome.org/CCID:008387] | |||
synonym: "BANF1-related neurodevelopmental syndrome" EXACT CLINGEN_LABEL [https://search.clinicalgenome.org/CCID\:008387] |
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.
@yshwetar did you make this change?
Please make sure the url is correct (ie without the " \ ")
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 did not - it has been showing up in my GH diff everytime I do a new PR
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 will update this when I resolve the conflict.
src/ontology/mondo-edit.obo
Outdated
name: severe combined immunodeficiency due to CTPS1 deficiency | ||
def: "Severe combined immunodeficiency (SCID) due to CTPS1 deficiency is a rare primary immunodeficiency disorder due to impaired capacity of activated T- and B-cells to proliferate in response to antigen receptor-mediated activation characterized by early-onset, severe, persistent and/or recurrent viral infections due to Epstein-Barr virus (EBV) and Varicella Zoster virus (VZV), (including generalized varicella), as well as recurrent sino-pulmonary bacterial infections due to encapsulated pathogens." [Orphanet:420573] | ||
name: combined immunodeficiency due to CTPS1 deficiency | ||
def: "Combined immunodeficiency due to CTPS1 deficiency is a rare primary immunodeficiency disorder due to impaired capacity of activated T- and B-cells to proliferate in response to antigen receptor-mediated activation characterized by early-onset, severe, persistent and/or recurrent viral infections due to Epstein-Barr virus (EBV) and Varicella Zoster virus (VZV), (including generalized varicella), as well as recurrent sino-pulmonary bacterial infections due to encapsulated pathogens." [https://orcid.org/0000-0002-0587-4693, Orphanet:420573, PMID:24870241] |
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.
You don't need to include "Combined immunodeficiency due to CTPS1 deficiency is" at the beginning of the definition (ie you don't need to repeat the term label)
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.
Please review the definition. Should it still include "severe"?
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.
Sorry, the previous definition had it that way except saying 'severe combined....'. But that makes sense! Thanks
src/ontology/mondo-edit.obo
Outdated
synonym: "IMD24" RELATED ABBREVIATION [MONDO:Lexical, OMIM:615897] | ||
synonym: "immunodeficiency 24" RELATED [MONDO:Lexical, OMIM:615897] | ||
synonym: "immunodeficiency type 24" EXACT [MONDORULE:2, OMIM:615897] | ||
synonym: "SCID due to CTPS1 deficiency" EXACT [Orphanet:420573] | ||
synonym: "SCID due to CTPS1 deficiency" RELATED [Orphanet:420573] | ||
synonym: "severe combined immunodeficiency due to CTPS1 deficiency" RELATED [] {source="https://orcid.org/0000-0002-0587-4693"} |
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.
synonyms require an x-ref, and not a source.
Question for you: is this disease "never" severe, or is it "sometimes" severe?
Depending on the answer to this question, you might need to update the scope of the synonym.
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.
From what I understand no. Hence why I put it as a related synonym. If it did sometimes present as severe, I would have done a narrow synonym.
Also I changed it to a x-ref!
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.
@yshwetar these are still showing as narrow synonyms, and "source" instead of x-ref.
And it is correct that they should be narrow synonym.
src/ontology/mondo-edit.obo
Outdated
@@ -331805,8 +331807,10 @@ xref: OMIM:615897 {source="Orphanet:420573", source="MONDO:equivalentTo", source | |||
xref: Orphanet:420573 {source="OMIM:615897", source="MONDO:equivalentTo"} | |||
xref: SCTID:763623001 {source="MONDO:equivalentTo"} | |||
xref: UMLS:C4014617 {source="MONDO:equivalentTo", source="MONDO:MEDGEN", source="MEDGEN:863054"} | |||
is_a: MONDO:0044201 {source="Orphanet:420573"} ! T+ B+ severe combined immunodeficiency |
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.
in addition to removing the SubClassOf annotation, you need to add an "excluded Subclass Of" annotation. See documentation here
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.
Addressed!
- Updated definition, removing the word severe, and the repeat of the label - Related synonym annotation changed from dbxref to a source - excluded subclass annotation added - Logical definition added
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.
@yshwetar I think some of your changes are not showing.
src/ontology/mondo-edit.obo
Outdated
@@ -309897,7 +309897,7 @@ subset: ordo_malformation_syndrome {source="Orphanet:280576"} | |||
subset: orphanet_rare {source="Orphanet:280576"} | |||
subset: otar {source="MONDO:OTAR"} | |||
subset: rare | |||
synonym: "BANF1-related neurodevelopmental syndrome" EXACT CLINGEN_LABEL [https://search.clinicalgenome.org/CCID:008387] | |||
synonym: "BANF1-related neurodevelopmental syndrome" EXACT CLINGEN_LABEL [https://search.clinicalgenome.org/CCID\:008387] |
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 will update this when I resolve the conflict.
src/ontology/mondo-edit.obo
Outdated
synonym: "IMD24" RELATED ABBREVIATION [MONDO:Lexical, OMIM:615897] | ||
synonym: "immunodeficiency 24" RELATED [MONDO:Lexical, OMIM:615897] | ||
synonym: "immunodeficiency type 24" EXACT [MONDORULE:2, OMIM:615897] | ||
synonym: "SCID due to CTPS1 deficiency" EXACT [Orphanet:420573] | ||
synonym: "SCID due to CTPS1 deficiency" RELATED [Orphanet:420573] | ||
synonym: "severe combined immunodeficiency due to CTPS1 deficiency" RELATED [] {source="https://orcid.org/0000-0002-0587-4693"} |
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.
@yshwetar these are still showing as narrow synonyms, and "source" instead of x-ref.
And it is correct that they should be narrow synonym.
src/ontology/mondo-edit.obo
Outdated
relationship: has_material_basis_in_germline_mutation_in http://identifiers.org/hgnc/2519 {source="MONDO:mim2gene_medgen"} ! CTPS1 | ||
property_value: excluded_subClassOf "'T+ B+ severe combined immunodeficiency'" xsd:string {source="https://orcid.org/0000-0002-0587-4693"} |
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.
you need to also have the original source (here: Orphanet:420573)
synonym: "SCID due to CTPS1 deficiency" EXACT [DOID:0111938, Orphanet:420573] | ||
synonym: "SCID due to CTPS1 deficiency" RELATED [] {source="Orphanet:420573"} | ||
synonym: "severe combined immunodeficiency due to CTPS1 deficiency" RELATED [] {source="https://orcid.org/0000-0002-0587-4693"} |
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.
@yshwetar please review these synonym scope and x-ref instead of "source" annotation
Closes #8178