Create go import #247
Merged
Create go import #247
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…of 'asexual reproduction', 'sexual repreduction', and 'viral life cycle' (all from GO and all now imported into SV), and replaced old GO reproduction with SV's new 'reproduction' everywhere it was used. Also fixed a reasoner error on 'contamination acquisition'. Updated all other imports in addition to creating new GO import.
…uded 'has output' relation from RO to update axiom for 'contamination acquisition'
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Updates to import GO more properly via ODK slme import.
Since SV used GO's 'reproduction' which is now obsolete, and because GO's 'reproductive process' is overly broad (the host's reproductive processes would qualify, and thus categorize any normal adult host as being infected in error), I also created class in SV called 'reproduction' which is the union of 'asexual reproduction' (to cover bacterial pathogens), 'sexual reproduction' (to cover eukaryotic pathogens that sexually reproduce), and 'viral life cycle' (to cover viral pathogens). We use the new reproduction class where the old GO one was used. We could have just left the class anonymous, but because we use it at least 4 or 5 times, it was much more convenient to "name" it.
Also found problems with class 'contamination acquisition' being not satisfiable due to use of has_specified_input and has_specified_output relations, which I changed to RO's 'has input' and 'has output'.
Fixes #246