-
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
Consider annotation prop to distinguish disease grouping class vs disease entity #685
Comments
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 |
Dear @nicolevasilevsky - did you hear back from EFO about this? |
@monicacecilia I don't think I ever did this! Thanks for the nudge |
per @paolaroncaglia's comments on EBISPOT/efo#553 (comment), I revised the subclass for MONDO_0005413 cystic fibrosis associated meconium ileum |
@nicolevasilevsky |
Got it - thanks @paolaroncaglia.
Note to self, these are the action items from EBISPOT/efo#553
pancreas disease
|
Is the original request in scope or feasible to be included in mondo, or should we look for a workaround? |
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. And based on the above, I'd also suggest to make 'cystic fibrosis associated meconium ileus' a subclass of MONDO:0054868 'meconium ileus' Thanks, |
Hi @nicolevasilevsky ,
|
Would use of github milestones help with prioritizing tickets for releases?
Kent can you say more about the negative consequence of classifying CF as
non leaf? I thought the distinction purely drove the kind of table
displayed, to minimize repetition with leaf terms
…On Wed, Nov 13, 2019, 05:28 paolaroncaglia ***@***.***> wrote:
Hi @nicolevasilevsky <https://github.com/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. Summing up:
- Edits (as applicable to Mondo) listed in #685 (comment)
<#685 (comment)>
- Edits suggested in #685 (comment)
<#685 (comment)>
Thank you very much.
Paola
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#685?email_source=notifications&email_token=AAAMMOICF7EOHZKAJ4M2IALQTP6H7A5CNFSM4HJNXBMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED6ENSI#issuecomment-553404105>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAMMOO5W2ODPBRRGKABQLLQTP6H7ANCNFSM4HJNXBMA>
.
|
@kshefchek - see comment above @cmungall yes- let's use GitHub milestones, great idea |
@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) |
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.
The text was updated successfully, but these errors were encountered: