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

Clarify definitions of navigational parent and child predicates #523

Closed
rjyounes opened this issue Jul 30, 2021 · 2 comments
Closed

Clarify definitions of navigational parent and child predicates #523

rjyounes opened this issue Jul 30, 2021 · 2 comments
Assignees
Labels
effort: small Requires less than one day to complete impact: patch No new functionality or changes in human-readable semantics (e.g,. fixing a typo in an annotation) status: implementation specified Implementation has been specified. A developer should be assigned.

Comments

@rjyounes
Copy link
Collaborator

gist 9.7.0 defines the following predicates (the latter two are for non-polyhierarchical trees):

hasSubCategory
hasNavigationalChild

hasSuperCategory
hasNavigationalParent

hasUniqueSuperCategory
hasUniqueNavigationalParent

I propose removing the navigation predicates, as I don't see any use for them that can't be handled with the category predicates. Judging from the names, they may have been created for faceting, but they still seem redundant to me.

@rjyounes rjyounes added impact: major Non-backward compatible (changes inferences; e.g., adding a restriction, domain, range) effort: small Requires less than one day to complete labels Jul 30, 2021
@rjyounes rjyounes added impact: minor New, backward-compatible functionality (does not change inferences; e.g., adding a term) impact: patch No new functionality or changes in human-readable semantics (e.g,. fixing a typo in an annotation) status: under review In triage and removed impact: major Non-backward compatible (changes inferences; e.g., adding a restriction, domain, range) impact: minor New, backward-compatible functionality (does not change inferences; e.g., adding a term) labels Sep 9, 2021
@rjyounes rjyounes added the status: implementation specified Implementation has been specified. A developer should be assigned. label Dec 16, 2021
@rjyounes rjyounes removed the status: under review In triage label Dec 17, 2021
@dylan-sa
Copy link
Contributor

dylan-sa commented Jan 4, 2022

In the 12/16 issue review meeting, we decided to keep the "navigational" predicates and tweak their skos:definitions to better indicate purpose. These predicates can be useful in some contexts (e.g., within a user interface) when the child category is not necessarily an instance of the parent category but it is nevertheless helpful to represent the relationship hierarchically. (For example, refrigerator handles aren't refrigerators, but it could be useful to group them under the refrigerator category.) Following up with a PR for this one shortly.

@rjyounes rjyounes changed the title Remove navigational parent and child predicates Clarify definitions of navigational parent and child predicates Jan 5, 2022
@uscholdm
Copy link
Contributor

This has been addressed. Current version is:

gist:hasNavigationalParent
	a owl:ObjectProperty ;
	skos:definition "Relates a child category to a parent category in an informal (e.g., faceted) hierarchy."^^xsd:string ;
	skos:example "Refrigerator handles are not refrigerators, but it may be useful to represent their relationship hierarchically for a faceted UI filter."^^xsd:string ;
	skos:prefLabel "has navigational parent"^^xsd:string ;
	.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: small Requires less than one day to complete impact: patch No new functionality or changes in human-readable semantics (e.g,. fixing a typo in an annotation) status: implementation specified Implementation has been specified. A developer should be assigned.
Projects
None yet
Development

No branches or pull requests

3 participants