peterjohnlawrence / com.inova8.odata2sparql.v4

Part of the overall odata2sparql solution, this odata2sparql.v4 module containing the olingo4 dependencies. In other words specifically for odata v4
http://inova8.com/bg_inova8.com/offerings/odata2sparql/
Apache License 2.0
5 stars 5 forks source link

Navigation property attachment #200

Closed radissoa closed 2 years ago

radissoa commented 3 years ago

Hello Peter,

You did a great job with odata2sparql. I'm looking for a small clarification on something. Here is the situation I am looking at : It looks like you attach a navigation property based on the rdfs:domain of the corresponding predicate, that means that for example you attach skos:broader navigation property to skos:Concept entity set. The thing is if I want to use skos:broader on another class, let's call :Item it is not attach to it. Note that apart from that :Item is a subclassOf skos:Concept.

Hence to get the broader concept of an item I am obligated to look for it through GET /skos_Concept('myItem')/skos_broader instead of GET /Item('myItem')/skos_broader which is more intuitive.

I don't think this is a bug and one solution could be to add my :Item as a rdfs:domain for skos:broader but this does not feel right and does not scale up when other situations like this occur for other properties in my ontology.

Do you have any better solution than mine for this issue ?

Best regards, Alexis

peterjohnlawrence commented 3 years ago

Alexis

If you make :Item a subclassOf skos:Concept then :Item should inherit all the navigation properties of a skos:Concept. Can you check in the $metadata that it is indeed a navigationproperty of :Item. If not we have a bug/configuration issue:-(

Peter J. Lawrence inova8 Providing answers for users' information questions Mobile: +44 7546 095977 | +1 330 631 3772 | Phone: +44 1298 872375 | Skype: PeterJLawrence Email: @.** | Web: www.inova8.com LinkedIn: *

http://www.linkedin.com/in/peterjohnlawrence http://www.linkedin.com/in/peterjohnlawrence

On Tue, 23 Mar 2021 at 14:29, Alexis Radisson @.***> wrote:

Hello Peter,

You did a great job with odata2sparql. I'm looking for a small clarification on something. Here is the situation I am looking at : It looks like you attach a navigation property based on the rdfs:domain of the corresponding predicate, that means that for example you attach skos:broader navigation property to skos:Concept entity set. The thing is if I want to use skos:broader on another class, let's call :Item it is not attach to it. Note that apart from that :Item is a subclassOf skos:Concept.

Hence to get the broader concept of an item I am obligated to look for it through GET /skos_Concept('myItem')/skos_broader instead of GET /Item('myItem')/skos_broader which is more intuitive.

I don't think this is a bug and one solution could be to add my :Item as a rdfs:domain for skos:broader but this does not feel right and does not scale up when other situations like this occur for other properties in my ontology.

Do you have any better solution than mine for this issue ?

Best regards, Alexis

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/peterjohnlawrence/com.inova8.odata2sparql.v4/issues/200, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMJVMC3M5TYKAJZBHACY6TTFCQUHANCNFSM4ZVJNM2Q .

radissoa commented 3 years ago

Sadly, the metadata does not show skos_broader as a navigation property of :Item.

peterjohnlawrence commented 3 years ago

Is it showing it as a navigationproperty of skos_concept?

Peter J. Lawrence inova8 Providing answers for users' information questions Mobile: +44 7546 095977 | +1 330 631 3772 | Phone: +44 1298 872375 | Skype: PeterJLawrence Email: @.** | Web: www.inova8.com LinkedIn: *

http://www.linkedin.com/in/peterjohnlawrence http://www.linkedin.com/in/peterjohnlawrence

On Wed, 24 Mar 2021 at 14:00, Alexis Radisson @.***> wrote:

Sadly, the metadata does not show skos_broader as a navigation property of :Item.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/peterjohnlawrence/com.inova8.odata2sparql.v4/issues/200#issuecomment-805845968, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMJVMEP5H6RCIU7P6MVUPDTFHV75ANCNFSM4ZVJNM2Q .

peterjohnlawrence commented 3 years ago

Can you share the $metadata with me, if that is possible. Thanks Peter J. Lawrence inova8 Providing answers for users' information questions Mobile: +44 7546 095977 | +1 330 631 3772 | Phone: +44 1298 872375 | Skype: PeterJLawrence Email: @.** | Web: www.inova8.com LinkedIn: *

http://www.linkedin.com/in/peterjohnlawrence http://www.linkedin.com/in/peterjohnlawrence

On Wed, 24 Mar 2021 at 14:02, Peter Lawrence @.***> wrote:

Is it showing it as a navigationproperty of skos_concept?

Peter J. Lawrence inova8 Providing answers for users' information questions Mobile: +44 7546 095977 | +1 330 631 3772 | Phone: +44 1298 872375 | Skype: PeterJLawrence Email: @.** | Web: www.inova8.com LinkedIn: *

http://www.linkedin.com/in/peterjohnlawrence http://www.linkedin.com/in/peterjohnlawrence

On Wed, 24 Mar 2021 at 14:00, Alexis Radisson @.***> wrote:

Sadly, the metadata does not show skos_broader as a navigation property of :Item.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/peterjohnlawrence/com.inova8.odata2sparql.v4/issues/200#issuecomment-805845968, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMJVMEP5H6RCIU7P6MVUPDTFHV75ANCNFSM4ZVJNM2Q .

radissoa commented 3 years ago

I sent it on your email. Alexis

peterjohnlawrence commented 2 years ago

This has now been fixed in #205. All superClass properties should have been inherited but there were certain cases when they were not