-
Notifications
You must be signed in to change notification settings - Fork 17
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
Changes to 'treatment' and 'prophylaxis' #98
Conversation
Moved ‘treatment’ and ‘prophylaxis’ to be subclasses of ‘planned process’. Changed definition of ‘treatment’ to reflect new superclass.
This reverts commit 3439b02.
to reflect Richard's definition in https://docs.google.com/spreadsheets/d/1RM6qZ_j27qn1E-jd1RFjDdhDjITKN-TR5Vb4DWlrw0Q as of 10 Nov 2015
to reflect Richard's definition in https://docs.google.com/spreadsheets/d/1RM6qZ_j27qn1E-jd1RFjDdhDjITKN-TR5Vb4DWlrw0Q as of 10 Nov 2015
And modified definition, changing ‘measurement assay’ to ‘assay’.
Also slight alteration to definition for ‘laboratory test’.
Addresses several outstanding issues, most of which are addressed here. This was partly done as an example of how proposed changes can be drafted in the OWL file directly by branching the master, modifying the ontology, and then requesting to have it pulled into the main branch when ready. A way for contributors to provide feedback without duplicating work. |
https://github.com/ShahimEssaid/git-owl-tools (local only) Moving forward, this should be solved with latest versions of Protege. You could roundtrip master than the branch and the diff should look ok. |
Moved ’treatment' and 'prophylaxis' to be subclasses of 'planned process’. Also changed the definition of 'treatment' to reflect new superclass.