Skip to content
This repository has been archived by the owner on Jan 21, 2021. It is now read-only.

TherapeuticProcedure misclassified #19

Open
harryhoch opened this issue Dec 11, 2017 · 2 comments
Open

TherapeuticProcedure misclassified #19

harryhoch opened this issue Dec 11, 2017 · 2 comments
Assignees

Comments

@harryhoch
Copy link
Contributor

TherapeuticProcedure contains a subclass restriction

subClassOf (hasRegimen some Medication Statement)

given that TherapeuticProcedure contains radiation therapies, surgeries, and endocrine therapy, suggest that we remove this restriction and place it on a more specific subclass. Reordering direct subclasses of TherapeuticRegimen might be appropriate as well.

@DavidHarris56
Copy link
Contributor

DavidHarris56 commented Dec 12, 2017 via email

@harryhoch
Copy link
Contributor Author

harryhoch commented Dec 13, 2017

2017-12-12_23-26-41
Dave, if you look at the entry for TherapeuticProcedure in Protege, there is a class restriction hasRegimen some MedicationStatement ( see attached image).

However, if you look at the subclasses of therapeutic procedure you see mastectomy, radiationtherapy, and other fields that don't include Medications, this restriction is not semantically sensible here.

I don't necessarily have a better solution - this whole subtree is a bit of a mess. If you're not sure how to fix it, let's table this until we get a chance to look at it in more detail.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants