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

Warnings from owlready on make all #281

Open
netfl0 opened this issue Aug 10, 2024 · 5 comments
Open

Warnings from owlready on make all #281

netfl0 opened this issue Aug 10, 2024 · 5 comments
Labels
bug Something isn't working
Milestone

Comments

@netfl0
Copy link
Contributor

netfl0 commented Aug 10, 2024

We think it might have something to do with our puns when you make all

#12 40.85 build/d3fend-architecture.owl done 
#12 41.39 build/d3fend-prefixes.json done 
#12 45.68 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2102
#12 45.68 
#12 45.68 * Owlready2 * Warning: ignoring cyclic type of, involving storid 843
#12 45.68 
#12 45.68 * Owlready2 * Warning: ignoring cyclic type of, involving storid 416
#12 45.68 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2511
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 722
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 723
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2531
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2385
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2154
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 842
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 855
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2109
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 889
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 753
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2123
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2319
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 907
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2697
#12 45.69 
#12 45.69 * Owlready2 * Warning: ignoring cyclic type of, involving storid 3776
#12 45.69 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2812
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2135
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2937
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2138
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2186
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 903
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2936
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2180
#12 45.70 
#12 45.70 * Owlready2 * Warning: ignoring cyclic type of, involving storid 2157
#12 45.70 ```

We wanted to migrate our puns from instances of their parent classes to instances of themselves. 

Any thoughts on this @hack-sentinel @aamedina ?

@netfl0 netfl0 added this to the 0.17.0 milestone Aug 10, 2024
@netfl0 netfl0 added the bug Something isn't working label Aug 10, 2024
@aamedina
Copy link
Collaborator

aamedina commented Aug 13, 2024

I think it has to do with asserting owl:Class, owl:NamedIndividual and the class itself ie

:SomeClass a owl:Class, owl:NamedIndividual, :SomeClass .

I think :SomeClass a :SomeClass can be removed and you can still use "punning". Not sure about how the GUI tools work though (eg protege et al).

My naive perspective is I think I'd prefer it to be unasserted just so I can query an ontology of individuals that transitively owl:imports d3fend for instances of a particular artifact class and not have it match the class itself without including that exception in the filter/query.

@netfl0
Copy link
Contributor Author

netfl0 commented Aug 13, 2024

To pun or not to pun...

We use the puns for the kb, and queries against the knowledge itself. Otherwise we had previously queries against restrictions.

@aamedina
Copy link
Collaborator

I definitely see the utility for metamodeling and appreciate the flexibility... If the tool knows how to deal with the semantics of punning, it seems powerful, I guess the "tricks" just need to be documented.

@netfl0
Copy link
Contributor Author

netfl0 commented Aug 13, 2024

Agree. Need to flesh out the Design.md

I forgot to note, the only tool having any issues with it is owlready2.

I believe our API and all the rest are handling puns correctly. I do believe the "instance of itself" is correct.

@aamedina
Copy link
Collaborator

Re: readmes I like the way FIBO documents their ontology conventions https://github.com/edmcouncil/fibo/blob/master/ONTOLOGY_GUIDE.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants