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

add new term mosaic SMO syndrome #7879

Closed
wants to merge 29 commits into from
Closed

add new term mosaic SMO syndrome #7879

wants to merge 29 commits into from

Conversation

twhetzel
Copy link
Collaborator

close #7768

@twhetzel twhetzel marked this pull request as draft June 28, 2024 07:09
@twhetzel twhetzel marked this pull request as ready for review June 29, 2024 21:45
@sabrinatoro sabrinatoro self-assigned this Jul 2, 2024
Copy link
Collaborator

@sabrinatoro sabrinatoro left a comment

Choose a reason for hiding this comment

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

@twhetzel There are a few things to change. Please let me know if you have questions.

xref: PMID:27236920 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:30497210 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:32413283 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:7606318 {source="https://clinicalgenome.org/affiliation/40059/"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

the PMID x-ref should only be used as x-ref for definition and synonym.
x-ref annotations to terms should be for x-ref to database or ontology

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

src/ontology/mondo-edit.obo Outdated Show resolved Hide resolved
xref: PMID:30497210 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:32413283 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:7606318 {source="https://clinicalgenome.org/affiliation/40059/"}
is_a: MONDO:0015160 {source="https://orcid.org/0000-0002-3458-4839"} ! multiple congenital anomalies/dysmorphic syndrome-variable intellectual disability syndrome
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you have any other source for this parent? If you made this parentage decision based on several internet searches, that's fine to add your orcid only, but it would be good to know how this parent was chosen (eg add the link to the google finds on the issue).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added this based on the disease description and PubMed references.

xref: PMID:32413283 {source="https://clinicalgenome.org/affiliation/40059/"}
xref: PMID:7606318 {source="https://clinicalgenome.org/affiliation/40059/"}
is_a: MONDO:0015160 {source="https://orcid.org/0000-0002-3458-4839"} ! multiple congenital anomalies/dysmorphic syndrome-variable intellectual disability syndrome
relationship: has_material_basis_in_germline_mutation_in http://identifiers.org/hgnc/11119 {source="https://clinicalgenome.org/affiliation/40059/"} ! SMO
Copy link
Collaborator

Choose a reason for hiding this comment

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

According to the issue, the mutation is not "germline". Please use 'has material basis in somatic mutation in'

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

is_a: MONDO:0015160 {source="https://orcid.org/0000-0002-3458-4839"} ! multiple congenital anomalies/dysmorphic syndrome-variable intellectual disability syndrome
relationship: has_material_basis_in_germline_mutation_in http://identifiers.org/hgnc/11119 {source="https://clinicalgenome.org/affiliation/40059/"} ! SMO
property_value: http://purl.org/dc/terms/creator https://orcid.org/0000-0002-3458-4839
property_value: IAO:0000233 "https://github.com/monarch-initiative/mondo/pull/7879" xsd:string
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be "xsd:anyURI" instead of "xsd:string"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

also updated to link to issue vs. PR

@sabrinatoro sabrinatoro assigned twhetzel and unassigned sabrinatoro Jul 8, 2024
src/ontology/mondo-edit.obo Outdated Show resolved Hide resolved
src/ontology/mondo-edit.obo Outdated Show resolved Hide resolved
src/ontology/mondo-edit.obo Outdated Show resolved Hide resolved
@twhetzel twhetzel assigned sabrinatoro and unassigned twhetzel Aug 22, 2024
katiermullen and others added 6 commits August 21, 2024 20:09
* Add non-human animal genetic diseases

addresses issue #7271

-adds more non-human animal diseases from OMIA,
-adds grouping terms where needed,
-adds db-xrefs to previously added non-human animal genetic diseaes

* update to fix QC violations

Updated to fix QC violations.

* update

* update

* update SCID in pigs

Combine terms, remove gene associations, add definition.

* addressed typos

---------

Co-authored-by: Sabrina Toro <[email protected]>
* Update mondo-edit.obo

* Add Mondo external content pipeline draft

* Update mondo.Makefile

* manual run of update-external-content make goal

---------

Co-authored-by: Nico Matentzoglu <[email protected]>
* MONDO:0022900 athyreotic congenital hypothyroidism obsolete

closes #7357

* update

delete unnecessary line
* MONDO:0000424 inborn vitamin B12 deficiency [Merge]

closes #7353

* update

add single child exclusion

* Update mondo-edit.obo
* add new term sycosis barbae

fixes #8081

* added information required in Mondo

---------

Co-authored-by: Sabrina Toro <[email protected]>
@twhetzel
Copy link
Collaborator Author

Closing because too many conflicts.

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.

[mosaic SMO syndrome/SMO]
5 participants