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

Deprecate isAspectOf, add hasAspect #603

Closed
rjyounes opened this issue Nov 30, 2021 · 7 comments
Closed

Deprecate isAspectOf, add hasAspect #603

rjyounes opened this issue Nov 30, 2021 · 7 comments

Comments

@rjyounes
Copy link
Collaborator

Normally the assertion and queries would go the other way: you don't want to know all the things that have a length aspect, you want to know what aspects a thing has. The former involves making assertions on a taxonomic term, which is odd.

Also make range of hasAspect gist:Aspect.

@uscholdm
Copy link
Contributor

uscholdm commented Dec 1, 2021

I agree with the main point.

Also make range of hasAspect gist:Aspect.

Makes sense at first glance, but this sort of thing tends to backfire as often as not. It might be we wans to say something hasAspect where the aspect is an aspect in everyday terms, but is strictly speaking not exactly what we mean by gist:Aspect. That can be done with SHACL.

@rjyounes
Copy link
Collaborator Author

Your point is a good one. I retract the range proposal.

@uscholdm
Copy link
Contributor

uscholdm commented Nov 9, 2023

@rjyounes
PROPOSAL:

  • Deprecate isAspectOf, add hasAspect
  • Don't have a domain or a range for hasAspect

This is timely, as the refactored units proposal (#697) will likely want to use hasAspect.

@rjyounes
Copy link
Collaborator Author

@uscholdm I agreem with this proposal. Should this issue be rolled into the units and measures model, or can we address it independently?

@uscholdm
Copy link
Contributor

Should this issue be rolled into the units and measures model, or can we address it independently?

Probably does not matter much - if somone want to do it now, not a bad idea. One less thing to document when UoM changes.

@rjyounes rjyounes self-assigned this Nov 30, 2023
@rjyounes rjyounes moved this from In Triage to To Do (typically assigned) in gist Version 12.1.0 Nov 30, 2023
@rjyounes rjyounes removed their assignment Nov 30, 2023
@rjyounes rjyounes moved this from To Do (typically assigned) to In Triage in gist Version 12.1.0 Nov 30, 2023
@rjyounes
Copy link
Collaborator Author

rjyounes commented Feb 8, 2024

@philblackwood Is this in accord with the new units and measures model?

@philblackwood
Copy link
Contributor

will be included in new Units of Measure.

@github-project-automation github-project-automation bot moved this from In Triage to Done in gist Version 13.0.0 Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

3 participants