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

Consider annotation prop to distinguish disease grouping class vs disease entity #685

Open
kshefchek opened this issue Apr 30, 2019 · 37 comments
Labels
effort-XL Ticket may take more than 2 hours complete low priority
Milestone

Comments

@kshefchek
Copy link
Contributor

It would be useful if MONDO included a way to distinguish between a disease entity and a disease grouping. Currently the only way to do this is to check for subclasses/types of a disease; however, this results in missing valid diseases (see Huntington disease and Cystic Fibrosis). We regularly add a check that all diseases with an OMIM equivalent are disease entities, but a more formal way to determine this would be useful for analyses and application views.

@cmungall
Copy link
Member

cmungall commented May 1, 2019

Not sure there is an agreed upon 'level'. E.g. ORDO calls juvenile Huntigton a disease entity

But we are doing something like this anyway for the mondo-analysis

I think the CF subclass is an error can you make ann efo ticket @nicolevasilevsky

@monicacecilia
Copy link
Contributor

Dear @nicolevasilevsky - did you hear back from EFO about this?

@nicolevasilevsky
Copy link
Member

@monicacecilia I don't think I ever did this! Thanks for the nudge

@nicolevasilevsky
Copy link
Member

per @paolaroncaglia's comments on EBISPOT/efo#553 (comment), I revised the subclass for MONDO_0005413 cystic fibrosis associated meconium ileum

@paolaroncaglia
Copy link
Collaborator

@nicolevasilevsky
MONDO:0009061 'cystic fibrosis' needs fixing too please, see
EBISPOT/efo#553 (comment)
Thanks!

@nicolevasilevsky
Copy link
Member

nicolevasilevsky commented Oct 10, 2019

Got it - thanks @paolaroncaglia.
I did one PR #882
I will work on the rest of this once @cmungall approves the PR.

  • Narrow down its disease location to 'intestine'

Note to self, these are the action items from EBISPOT/efo#553

pancreas disease
Rare genetic respiratory disease
rare male fertility disorder with obstructive azoospermia
Rare genetic disorder with obstructive azoospermia
rare pulmonary disease
Genetic biliary tract disease
Genetic pancreatic disease

@kshefchek
Copy link
Contributor Author

Is the original request in scope or feasible to be included in mondo, or should we look for a workaround?

@paolaroncaglia
Copy link
Collaborator

Going back to MONDO:0005413 'cystic fibrosis associated meconium ileum', I'm afraid there's a typo in the label that you inherited from EFO, it should be "ileus" not "ileum". (See e.g.
https://www.cysticfibrosisjournal.com/article/S1569-1993(17)30809-3/fulltext
https://www.chop.edu/conditions-diseases/meconium-ileus
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3085752/)

And based on the above, I'd also suggest to make 'cystic fibrosis associated meconium ileus' a subclass of MONDO:0054868 'meconium ileus'

Thanks,
Paola

@paolaroncaglia
Copy link
Collaborator

paolaroncaglia commented Nov 13, 2019

Hi @nicolevasilevsky ,
I'm afraid that part of this ticket got a bit side-tracked due to technical issues ;-)
It would be great if you could please, before your next release, complete the edits agreed upon for 'cystic fibrosis', so EFO can resume mapping to Mondo for this term. Summing up:

@cmungall
Copy link
Member

cmungall commented Nov 13, 2019 via email

@nicolevasilevsky
Copy link
Member

@kshefchek - see comment above

@cmungall yes- let's use GitHub milestones, great idea

@nicolevasilevsky nicolevasilevsky added this to the December release milestone Nov 13, 2019
@nicolevasilevsky
Copy link
Member

@paolaroncaglia, I assigned this ticket to the December release milestone (which I just created).

Are you able to assign milestones too? Feel free to assign and help prioritize. (If not, I think I can adjust your permissions so you can do so)

@nicolevasilevsky nicolevasilevsky added this to the March release milestone Oct 22, 2021
@nicolevasilevsky nicolevasilevsky removed their assignment Jan 4, 2024
@sagehrke sagehrke moved this to Backlog in 2024-02 Mondo Release Jan 4, 2024
@sagehrke sagehrke moved this to Backlog in 2024-03 Mondo Release Feb 22, 2024
@sabrinatoro sabrinatoro moved this to Backlog in 2024-04 Mondo Release Mar 4, 2024
@sabrinatoro sabrinatoro moved this from Backlog to Needs Discussion in 2024-04 Mondo Release Mar 28, 2024
@sabrinatoro sabrinatoro moved this from Backlog to Needs Discussion in 2024-05 Mondo Release Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort-XL Ticket may take more than 2 hours complete low priority
Projects
None yet
Development

No branches or pull requests

9 participants