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

MONDO:0037821 porphyrin metabolism disease[Revise subclass] #7381

Closed
kanems opened this issue Mar 6, 2024 · 3 comments · Fixed by #8206
Closed

MONDO:0037821 porphyrin metabolism disease[Revise subclass] #7381

kanems opened this issue Mar 6, 2024 · 3 comments · Fixed by #8206
Assignees
Labels
Revise subclass user request A request from an external user

Comments

@kanems
Copy link
Collaborator

kanems commented Mar 6, 2024

Mondo term (ID and Label)
MONDO:0037821 porphyrin metabolism disease

Suggested revision and reasons
I am not sure I understand the current structure/relationships in this branch.
I see inborn porphyrin metabolism disorders, but no acquired forms. Then porphyria has hepatic and inborn forms, but are the hepatic forms of porphyria all acquired?
SNOMED CT 29094004 (as shown in UMLS) has a number of children that don't seem to be located under this part of Mondo's hierarchy (I am adding a couple of Mondo IDs that are potential matches, but I know this needs more in-depth research)
image
Disorder of porphyrin metabolism
-Disorder of bilirubin metabolism Could this match MONDO:0024431 ?
-Ferrochelatase deficiency MONDO:0008319 ?
-Heme oxygenase-1 deficiency
-Inherited disorder of porphyrin metabolism
-Neonatal jaundice with porphyria
-Porphobilinogen deaminase deficiency
-Porphyria
-Porphyrinopathy
-Porphyruria
-Uroporphyrinogen decarboxylase deficiency

I see UMLS has lumped porphyrin & heme metabolism disorders with porphyria, so I will ask if they can review this mapping to better align with SNOMED and we will update MedGen's CUI mappings accordingly.

Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

@twhetzel
Copy link
Collaborator

@kanems we are working on this now and yes, there should be at least an 'acquired porphyria' since 'porphyria cutanea tarda' has both an acquired and inherited form. I will continue reviewing and sort out more details.

A question for you, it looks like MONDO:0022714 'chester porphyria' should be obsoleted and merged with MONDO:0008294 'acute intermittent porphyria'. Do you agree with this change?

@kanems
Copy link
Collaborator Author

kanems commented Sep 26, 2024

@twhetzel I would not merge the Chester-type porphyria concept with AIP for a few reasons...

  1. OMIM treats it as an "included" concept https://www.omim.org/entry/176000
  2. OMIM's description of the initial family with the disorder indicates it's highly variable phenotype, though molecularly it is a "type" of AIP since the same gene is affected as AIP.
  3. UMLS, MeSH, and SNOMED CT have unique codes for it (UMLS CUI: C0268322| MeSH C536341| SNOMEDCT_US 84816006 )

@twhetzel
Copy link
Collaborator

twhetzel commented Sep 27, 2024

Thanks @kanems, we did not propose to merge those terms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Revise subclass user request A request from an external user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants