-
Notifications
You must be signed in to change notification settings - Fork 69
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
GitHub-2039 - Implicit punning of OWL Classes in triples (as objects) with object property #2049
Conversation
…rate actions to eliminate punning issues Signed-off-by: Elisa Kendall <ekendall@thematix.com>
… 500 to correct implicit punning issues Signed-off-by: Elisa Kendall <ekendall@thematix.com>
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.
It is quite an unintuitive modeling approach to have restrictions on Individuals, and I would need to understand the rationale.
Worse, it seems to have a flaw - use of someValuesFrom requires at least one instance to be classified by any classifier.
@rivettp It is perfectly legal to do this, and we used this pattern quite successfully at Wells Fargo and on other projects such as IDMP. It is needed when an individual classifies something when it applies to all members of that class. We originally applied this approach for the CFI classification scheme, but agreed to change that given that every organization uses the CFI classification and it is now mandated by the FDTA. Many organizations do not use ISO 15022 to classify corporate actions, however, and we would be mandating its use in all cases if we follow the pattern used for the CFI. If people are using this ontology then they are committing to classifying corporate actions using the ISO 15022 classification scheme, and so the at least one corporate action classified by that scheme is entirely appropriate. |
@rivettp What we had previously caused punning, so either we do this, or we add restrictions on the elements of the two ontologies that represent the corporate actions in this local ontology. That's less intuitive IMHO. |
You didn't address my second comment that it requires at least one instance of any action. |
@rivettp It does, but using an open world assumption one would say that there is an instance of such an action somewhere in the universe but we may not know about it. Also perfectly legal. It would only be required if you close the world. |
@rivettp The other option would be to change it to either all values from or min 0. I can make that change, but all values from might be problematic for people wanting to use the ACTUS classifiers. |
why not allValuesFrom? More generally if this is a pattern we think is a good idea here, are there not other places we should review for where it should be applied? |
@rivettp I would have to try it with examples, using two classifiers in different schemes for the same thing, such as a repurchase offer, then run the reasoner to see what happens. It might make a logical equivalence between the two, but that might be ok if they are truly equivalent. With some values from, that leaves the door open for some actions to not be classified by a particular code. |
The restriction would apply only to an individual code such as BONU and declares it's a member of the class of things that classify only instances of BonusIssue. |
…estrictions related to classifying actions Signed-off-by: Elisa Kendall <ekendall@thematix.com>
7ac4943
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.
Happier with the allValuesFrom
Description
Revised the modeling strategy for ISO 15022/20022 corporate actions and example market indices baskets to eliminate punning issues
Fixes: #2039
Checklist: