-
Notifications
You must be signed in to change notification settings - Fork 54
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
revise synonyms for 'developmental and epileptic encephalopathy, 31A' #8605
Conversation
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.
@nicolevasilevsky I left some comments for you. Some of them are for my own knowledge. Thank you so much!
@@ -558450,6 +558452,7 @@ name: developmental and epileptic encephalopathy, 31B | |||
def: "Any developmental and epileptic encephalopathy in which the cause of the disease is a homozygous mutation in the DNM1 gene." [https://orcid.org/0000-0002-0587-4693, OMIM:620352] | |||
subset: gard_rare {source="MONDO:GARD"} | |||
subset: rare | |||
synonym: "developmental and epileptic encephalopathy 31B, autosomal recessive" EXACT [OMIM:620352] | |||
synonym: "developmental and epileptic encephalopathy 31B, autosomal recessive" EXACT [https://orcid.org/0000-0001-9310-0163] |
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.
Perhaps combine these above two lines?
Also this syn was not requested in the ticket, but I did find that OMIM lists the syn DEE31B.
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.
@nicolevasilevsky what do you think about adding the exact syn DEE31B to this term for completeness? Source is OMIM:620352.
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.
great idea, will do
this is ready for re-review @katiermullen |
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.
@nicolevasilevsky I approve this PR. I just wonder if the exact syn DEE31B should be added to term MONDO:0957248 before merging? Thank you for all of your work on these issues!
@@ -558450,6 +558452,7 @@ name: developmental and epileptic encephalopathy, 31B | |||
def: "Any developmental and epileptic encephalopathy in which the cause of the disease is a homozygous mutation in the DNM1 gene." [https://orcid.org/0000-0002-0587-4693, OMIM:620352] | |||
subset: gard_rare {source="MONDO:GARD"} | |||
subset: rare | |||
synonym: "developmental and epileptic encephalopathy 31B, autosomal recessive" EXACT [OMIM:620352] | |||
synonym: "developmental and epileptic encephalopathy 31B, autosomal recessive" EXACT [https://orcid.org/0000-0001-9310-0163] |
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.
@nicolevasilevsky what do you think about adding the exact syn DEE31B to this term for completeness? Source is OMIM:620352.
close #8019