semanticarts / gist

Semantic Arts gist upper enterprise ontology
Creative Commons Attribution 4.0 International
160 stars 18 forks source link

gist:hasPhysicalLocation #940

Closed uscholdm closed 9 months ago

uscholdm commented 1 year ago
gist:hasPhysicalLocation
    a
        owl:ObjectProperty ,
        owl:TransitiveProperty
        ;
    rdfs:range gist:Place ;
    skos:definition "Where something is located"^^xsd:string ;
    skos:prefLabel "has physical location"^^xsd:string ;
    .

I propose changing the skos definition to:

ALSO: I propose adding a subclass restriction to the class, gist:PhysicalIdentifiableItem requiring it to have a physical location. Ie. (gist:hasPhysicalLocation some gist:Place). Perhaps the same for gist:PhysicalSubstance

rjyounes commented 1 year ago

How about: "Relates something to its physical location"?

The restriction seems good.

Didn't we also want a scope note to the effect that the location could be either permanent (a house) or transient (a car), and that if someone wants to specify a transient relationship they should use a temporal relation?

uscholdm commented 1 year ago

How about: "Relates something to its physical location"?

A definite improvement. Good idea.

uscholdm commented 1 year ago

Didn't we also want a scope note to the effect that the location could be either permanent (a house) or transient (a car), and that if someone wants to specify a transient relationship they should use a temporal relation?

We could, in which case, how about:

skos:scopeNote "To model changing locations, create a subclass of TemporalRelation (e.g. EstablishedLocation)" that reifies hasPhysicalLocation^^xsd:string ; 

This might be setting a precedent; we don't usually say how to extend gist, that could get out of hand quickly. But it might make sense in this case. Let's see what people think.

uscholdm commented 1 year ago

Updated proposal

gist:hasPhysicalLocation
    a
        owl:ObjectProperty ,
        owl:TransitiveProperty
        ;
    rdfs:range gist:Place ;
    skos:definition "Relates something to its physical location."^^xsd:string ;
    skos:prefLabel "has physical location"^^xsd:string ;
    skos:scopeNote "To model changing locations, create a subclass of TemporalRelation (e.g. EstablishedLocation) that reifies hasPhysicalLocation"^^xsd:string ; 
    .
rjyounes commented 1 year ago

Oddly, EstablishedLocation sounds to me like the permanent version.

Maybe we don't have to give explicit suggestions. How about just:

"This property includes both temporary (e.g., a car) and permanent (e.g., a house) locations of objects."

uscholdm commented 1 year ago

Given this was closely related to PR #941, I updated hasPhysicalLocation as part of that PR. It currently is:

gist:hasPhysicalLocation
    a
        owl:ObjectProperty ,
        owl:TransitiveProperty
        ;
    rdfs:range gist:Place ;
    skos:definition "Relates something to its physical location."^^xsd:string ;
    skos:prefLabel "has physical location"^^xsd:string ;
    skos:scopeNote "This property does not distinguish between things whose locations are stable and those whose locations change over time; e.g., a fire hydrant vs. a car."^^xsd:string ;
    .
rjyounes commented 9 months ago

Closing - fixed in PR #941.