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

Rename TemporalRelation to TemporalRelationship #478

Closed
uscholdm opened this issue May 18, 2021 · 6 comments
Closed

Rename TemporalRelation to TemporalRelationship #478

uscholdm opened this issue May 18, 2021 · 6 comments

Comments

@uscholdm
Copy link
Contributor

An instance of gist:TemporalRelation is a relationship between two things, it is not a relation. So the class should be renamed to TemporalRelationship

See also #188

@rjyounes
Copy link
Collaborator

What's the difference between a relation and a relationship? Relation: "the way in which two or more concepts, objects, or people are connected..." Why is this not appropriate?

@uscholdm
Copy link
Contributor Author

uscholdm commented May 26, 2021

The way I am related to you is the same way that Dave is related to you (being a work colleague of) isColleagueOf. OWL calls that a property, in math/logic it is called a relation. There is just one relation, but many relationships. Here are two:

  • Michael is colleague of Dave
  • Rebecca is colleague of Dave

These are relationSHIPs, not relaSHINS. The time stamps are different for each of these two instances of (what should be called) TemporalRelationship. Or more likely, a subclass of TemporalRelationship whose skos:definition might be "collegial relationship between two people that holds for a period of time."

@rjyounes
Copy link
Collaborator

I see. That makes sense.

@uscholdm
Copy link
Contributor Author

uscholdm commented Sep 7, 2023

@rjyounes @DanCarey404
We should revisit this, and if we keep it, it should ideally be along with other name changes. It's minor and could be tempered with a scope note. Dan often wished we had named it TemporalRelationship

@rjyounes
Copy link
Collaborator

I've added it to the 12.1.0 project for review.

@rjyounes
Copy link
Collaborator

Closing - not significant enough to change.

@rjyounes rjyounes closed this as not planned Won't fix, can't repro, duplicate, stale Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants