Skip to content
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

Merged
merged 3 commits into from
Sep 20, 2024

Conversation

ElisaKendall
Copy link
Contributor

@ElisaKendall ElisaKendall commented Sep 18, 2024

Description

Revised the modeling strategy for ISO 15022/20022 corporate actions and example market indices baskets to eliminate punning issues

Fixes: #2039

Checklist:

  • I'm familiar with the FIBO developer quide. My contribution meets all the requirements described there.
  • My contribution follows the principles of best practices for FIBO.
  • My changes have been reconciled with latest master and no merge conflicts remain.
  • This PR is related to exactly one issue. The issue is referenced by using a GitHub keyword such as "fixes", "closes", or "resolves".
  • Hygiene tests have been applied by a PR with "(WIP)" in title.
  • The issue has been tested locally using a reasoner (for ontology changes).

…rate actions to eliminate punning issues

Signed-off-by: Elisa Kendall <ekendall@thematix.com>
@ElisaKendall ElisaKendall added bug CAE Corporate Actions and Events Domain labels Sep 18, 2024
@ElisaKendall ElisaKendall self-assigned this Sep 18, 2024
… 500 to correct implicit punning issues

Signed-off-by: Elisa Kendall <ekendall@thematix.com>
@ElisaKendall ElisaKendall changed the title GitHub-2039 - Implicit punning of OWL Classes in triples (as objects) with object property (WIP) GitHub-2039 - Implicit punning of OWL Classes in triples (as objects) with object property Sep 19, 2024
jfgemski
jfgemski previously approved these changes Sep 19, 2024
tahoeblue
tahoeblue previously approved these changes Sep 19, 2024
Copy link
Contributor

@rivettp rivettp left a 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.

@ElisaKendall
Copy link
Contributor Author

@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.

@ElisaKendall
Copy link
Contributor Author

@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.

@rivettp
Copy link
Contributor

rivettp commented Sep 19, 2024

You didn't address my second comment that it requires at least one instance of any action.

@ElisaKendall
Copy link
Contributor Author

@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.

@ElisaKendall
Copy link
Contributor Author

@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.

@rivettp
Copy link
Contributor

rivettp commented Sep 19, 2024

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?

@ElisaKendall
Copy link
Contributor Author

@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.

@rivettp
Copy link
Contributor

rivettp commented Sep 19, 2024

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.
I think.
The fact that it's hard to think through, and even you feel the need to discover what happens in a reasoner, is a bit of a warning sign that we need to tread carefully with this pattern. I think something that's simple and obvious may be more important than an extra level of checking. Especially since there are many checks we forget about even trying due to reasoner performance, even for toy examples let alone enterprise data. Not to mention for DL safe reasons.
And we shouldn't forget about the possibility of expressing constraints using SHACL - which is what it's designed for.

…estrictions related to classifying actions

Signed-off-by: Elisa Kendall <ekendall@thematix.com>
Copy link
Contributor

@rivettp rivettp left a 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

@ElisaKendall ElisaKendall merged commit bd3c263 into edmcouncil:master Sep 20, 2024
4 checks passed
@ElisaKendall ElisaKendall deleted the GitHub-2039 branch September 20, 2024 00:24
@mereolog mereolog added this to the 2024Q3 FIBO Release milestone Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug CAE Corporate Actions and Events Domain
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implicit punning of OWL Classes in triples (as objects) with object property
5 participants