-
Notifications
You must be signed in to change notification settings - Fork 7
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
Harmonize raman spectroscopy with obi assay pattern #129
Open
StroemPhi
wants to merge
20
commits into
main
Choose a base branch
from
Harmonize-Raman-Spectroscopy-with-obi-assay-pattern
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 9 commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
3b9ffb3
Add equivalent class axiom column
StroemPhi 4f9f31e
Add main Raman scattering effects according to ISO
StroemPhi 26214a3
Make Stokes & anti-Stokes Raman scattering disjoint
StroemPhi 501eea9
Merge branch 'main' into Add-Raman-spectroscopy-objective-patterns
StroemPhi 031bf11
Delete general class axiom that got accidentally checked in
StroemPhi e43a296
Add Stokes & anti-Stokes Raman spectroscopy subclasses
StroemPhi b4b26f2
Inject subclassOf axioms on CHMO_0000656 accodring to OBI:assay pattern
StroemPhi b090e4c
Update TBox visualization
StroemPhi 4f17909
Remove redundant subclassOf axioms
StroemPhi 34270b0
Add forgotton specified input axioms to Stokes & anti-Stokes Raman sp…
StroemPhi 3d4307d
Update vibso_classes.owl
Zack-83 470e67b
Merge branch 'Harmonize-Raman-Spectroscopy-with-obi-assay-pattern' of…
StroemPhi e245fbe
Fix typo in definition of Raman raw data
StroemPhi 0cb9725
Adding reference to the introducing PR
StroemPhi 2fee1a0
Merge branch 'main' into Harmonize-Raman-Spectroscopy-with-obi-assay-…
Zack-83 8cc81a1
beautify table
Zack-83 8105cc0
fix "measurment"
Zack-83 883de50
Fix ID mixup after updating from main
StroemPhi c636872
Sort columns according to #145
Zack-83 351c1d6
Merge branch 'main' into Harmonize-Raman-Spectroscopy-with-obi-assay-…
Zack-83 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The right panel is somehow readable for someone who understands OWL. But yet the definition looks too complex. "Raman scattering" and "Raman spectroscopy" are really too far from each other. In my opinion, we should introduce once an object property "technique implements phenomenon" and then include in it all restrictions.
The left panel does not help at all, sorry. Apparently such a graphical representation is not adapt to capture che complexity of multiple restrictions defined through unnamed nodes. So better leave it away, or replace it with something less picky but more understandable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the right panel comment:
While I agree that it looks complicated for someone not versed in OWL, this proposal follows the OBI pattern (see also: https://nfdi4chem.github.io/VibrationalSpectroscopyOntology/design_patterns/#defining-an-assay-according-to-the-obi-core-assay-pattern) and since CHMO is an OBI extentsion and VIBSO is planned as a CHMO extention, we must follow the design patterns of OBI.
But this doesn't mean I'm objecting to defining such an object property to make things easier for those kinds of assays that can be defined as methods that cause and detect a natural process (effect). As written in the notes of our last dev call importing SSN:detects won't be suitable for this. To be in line with OBO a possible parent for such a property could be RO:causally upstream of or within, which connects two causally related processes.
However, we still would want to have axioms that specify what kind of input and output such an assay has. Hence, formalizing that the evaluated material entity in such an assay is the cause of the detected natural effect seems important to me. To reduce the complexity on that part, we could define for example a "Raman spectroscopy evaluant" class as a
and then just assert on Raman spectroscopy:
has_specified_input some 'Raman spectroscopy evaluant'
.And we could do so similarly for the data item produced by a Raman spectroscopy, which we might want to label "Raman spectral data" and which would serve as parent for all types of data outputs of Raman spectroscopies, the unprocessed ones, like "Raman raw data", as well as the ones produced by a post-processing step, such as the plot of the spectrum.
Regarding the left panel comment:
I don't get what you mean. It is of course a reduction, but it contains as nodes and edges all asserted and inherited axioms of Raman spectroscopy. Hence it was only meant to illustrate the pattern, which imho makes it easier to see the similarities/closeness to the OBI:assay pattern.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about using as direct relation between Raman spectroscopy and the Raman scattering it detects RO:regulates?
It is defined as a causal relation between processes:
So we could assert the subclassOf axiom: