obi icon indicating copy to clipboard operation
obi copied to clipboard

CFDE NTRs for HuBMAP project

Open mgiglio99 opened this issue 3 years ago • 14 comments

Hello, We are working on Batch 2 of term requests as part of the CFDE project. This set is needed by the HuBMAP project. Here is a list of the terms with draft definitions, parentage, etc. https://docs.google.com/spreadsheets/d/1_XaxKYQKO0nnH8yY8zwz_lhkweuUsO0ARyF22PI1uTk/edit?usp=sharing We are working on finalizing them, but I feel the terms are far enough along to reserve OBI term ids. Please take a look and see if you agree that they are in scope for OBI and that it would be ok to register OBI ids for them at this stage. Thanks, Michelle and @nsuvarnaiari

mgiglio99 avatar Jul 19 '21 14:07 mgiglio99

Hi @mgiglio99,

here is the definition for 'detection technique' I showed in the OBI call on 7/26/21.

Detection technique =(def) Assay D counts as a detection technique in assay A if and only if

(I) In generating information ia about target ta of A, one utilizes D to generate information id about a proxy td, where td is the target of D and id is used to determine what a measurement of ia would be. And

(II) td and ta are disjoint.

in looking at the terms you requested:

  1. what you have for row 10 as a detection technique seems fine to me. It is the fluorescence detection assay that detects the fluorescently labeled molecules to get at localizing specific DNA or RNA sequences.
  2. Row 7 doesn't have a textual definition, but I would imagine (given the label) that identifying an antibody for imaging will be done in virtue of detecting fluorescence.
  3. Row 4 is an interesting case. Because the parent for "autofluorescence microscopy assay" is "autofluorescence microscopy assay" (which has as a parent "fluorescence detection assay"). My intuition is to leave "detection technique" empty here. Because while a microscope is used to produce a magnified image of the object, I'm assuming the object is still detected via a fluorescence detection assay. (but I could be wrong).
  4. for row 18 I am also not sure. This may be fine. sometimes it seems to me that mass spectrometry assays are detection techniques and other times not. I'm not sure what to think in this case. It seems to work here.

I hope this helps

hectorguzor avatar Jul 26 '21 18:07 hectorguzor

Thanks for this Hector - I'll take a look at these more closely - this is great input as we are still working on developing these. Another set that we've already done the pull request for and that we feel are ready to and that we'd like to get your input on are in https://github.com/obi-ontology/obi/issues/1243 Could you take a look at those ? Thanks, Michelle

mgiglio99 avatar Jul 27 '21 21:07 mgiglio99

@hectorguzor We are finally getting around to reviewing these more closely. We've managed to review the first 3 so far. For number 2, thanks for pointing that out - we've added a def - not sure how we missed adding one before. For number 3, we agree about removing the 'fluorescence microscopy' detection technique. We are thinking that we will add equivalence axioms from 'microscopy assay' instead so that this term will get an inferred microscopy parent. One thing we considered was being able to add equivalence axioms from 'fluorescence microscopy' so as to be more specific, but that term is from CHMO and doesn't have equivalence axioms. Do you think we should consider making an OBI term for that? Thanks, Michelle

mgiglio99 avatar Oct 25 '21 18:10 mgiglio99

@mgiglio99 I see what you mean for number 3. It would be great if the CHMO term had an equivalence axiom. I'm actually not sure how to proceed here. Creating a new term may work, but then would duplicate terms. This is worth discussing on an OBI call.

If you are not committed to having "autofluorescence assay" as a parent, the other option could be to make the CHMO term the parent, create an equivalence axiom for "autofluorescence assay" (if possible), and use that to get the inference you want.

hectorguzor avatar Oct 25 '21 19:10 hectorguzor

Discussed on Nov. 1 call - CHMO term has been adopted by OBI for long term care, therefore it’s in the assay template and we can change it as needed. We will add equivalence axioms to it and then use it as an inferred parent of our new ‘autofluorescence microscopy’ term.

mgiglio99 avatar Nov 01 '21 16:11 mgiglio99

Suvvi and I think that all of the remaining questions for this batch of NTRs have been resolved. Here is the final spreadsheet for review: https://docs.google.com/spreadsheets/d/1_XaxKYQKO0nnH8yY8zwz_lhkweuUsO0ARyF22PI1uTk/edit?usp=sharing We hope to finalize these at the next OBI call.

mgiglio99 avatar Nov 12 '21 20:11 mgiglio99

Hi @hectorguzor

We would like to change the label for "fluorescence microscopy" (CHMO:0000087) to "fluorescence microscopy assay" . @jamesaoverton gave us an "ok" for this change, so went ahead and revised the name in the google doc below.

Could you please check the equivalence axiom for "fluorescence microscopy assay" in the google doc - https://docs.google.com/spreadsheets/d/1_XaxKYQKO0nnH8yY8zwz_lhkweuUsO0ARyF22PI1uTk/edit?usp=sharing

"fluorescence microscopy" is a child of "microscopy assay" (OBI:0002119) and we want to add "fluorescence detection assay" (OBI:0001501) an inferred parent.

Also, please take a look at other terms which are complete and ready for submission. If everything looks good, we will go ahead with a PR.

Thanks, Suvvi

nsuvarnaiari avatar Nov 18 '21 21:11 nsuvarnaiari

Hi @nsuvarnaiari

  • adding '('material entity' and ('bearer of' some fluorescence))' to the input column of 'fluorescence microscopy' gets you 'fluorescence detection assay' as an inferred parent.
  • 'fluorescence detection assay' can be removed from the detection technique column. Since it is an inferred parent 'fluorescence microscopy' wouldn't need to have it as a part.
  • 'microscope' can be removed from the devise column. It already inherits this from 'microscopy assay'

Yes, I will take a look at the other terms so we can put this on the OBI agenda for 2021-11-22 to get that PR going. Hector

hectorguzor avatar Nov 18 '21 23:11 hectorguzor

Hi @nsuvarnaiari I got a chance to look at the other assays.

  • For 'autofluorescence microscopy assay' (row 5) you don't need '('material entity' and ('bearer of' some fluorescence))' because it is already a type of 'fluorescence detection assay'.

  • For 'fluorescence in-situ hybridization assay' (row 11) OBI has chromosome organization assay by fluorescence in-situ hybridization and this has 'FISH' as an alternative term. Are these the same?

Everything else looks good to me. Let me know if you have any questions.

hectorguzor avatar Nov 19 '21 18:11 hectorguzor

Discussed on OBI call 2021-11-22: There was concern that making 'fluorescence microscopy' an equivalence class that has ('material entity' and ('bearer of' some fluorescence)) would unintentionally infer children of 'fluorescence microscopy' into 'fluorescence detection assay'. Hector will check in protege to make sure this is not the case. It was also suggested to make 'light microscopy' the asserted parent of 'fluorescence microscopy'.

hectorguzor avatar Nov 22 '21 18:11 hectorguzor

Getting 'fluorescence detection assay' as an inferred parent for 'fluorescence microscopy' was possible by making the 'logical type' a 'subclass' (if making it 'equivalent' is a concern). Here is what it looks like

fluro-micro

Using robot diff, I checked that adding ('material entity' and ('bearer of' some fluorescence)) did not generate any weird inferences. Everything looks good. This is all I got with the diff report.

F-M

I made 'light microscopy' the asserted parent as discussed in OBI call 2021-11-22. 'fluorescence microscopy' got inferred as a child of 'fluorescence detection assay'. You don't get, for example, 'widefield microscopy' as a child of 'fluorescence detection assay' (I believe that was the concern).

hectorguzor avatar Nov 22 '21 23:11 hectorguzor

Hi @cstoeckert , @bpeters42 and @hectorguzor

Thanks Hector for checking the equivalence axiom for 'fluorescence microscopy'. Glad everything is looking good. If nobody has any other concerns we will go ahead with the pull request.

Thanks, Suvvi and Michelle

nsuvarnaiari avatar Nov 29 '21 16:11 nsuvarnaiari

Approved for pull request step on Nov 29 call.

ddooley avatar Nov 29 '21 17:11 ddooley

Michelle and I made another change in the template after going over Hector's comment from Nov 19th.

Currently, the ontology has chromosome organization assay by fluorescence in-situ hybridization a child of in-situ hybridization assay, but we are proposing it to be a child of 'fluorescence in-situ hybridization assay' instead, a new term in the template (row 11). We are also proposing "FISH" to be an alternate term for 'fluorescence in-situ hybridization assay' but not chromosome organization assay by fluorescence in-situ hybridization.

Template: https://docs.google.com/spreadsheets/d/1_XaxKYQKO0nnH8yY8zwz_lhkweuUsO0ARyF22PI1uTk/edit?usp=sharing

Hope this edit is making sense. Thanks, Suvvi

nsuvarnaiari avatar Dec 16 '21 21:12 nsuvarnaiari

Hi @mgiglio99, I believe this issue has been resolved. If it has not, we can reopen it. But I will go ahead and close it for now.

hectorguzor avatar Mar 09 '23 16:03 hectorguzor