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

ore:aggregates (wiki) vs pcdm:hasRelatedObject (ontology) #48

Open
azaroth42 opened this issue Apr 14, 2016 · 7 comments
Open

ore:aggregates (wiki) vs pcdm:hasRelatedObject (ontology) #48

azaroth42 opened this issue Apr 14, 2016 · 7 comments

Comments

@azaroth42
Copy link
Contributor

... which is correct?

I'm in favor of pcdm:hasRelatedObject. I think this was the decision way back when, but in moving the docs around from pillar to post, they seem to have reverted.

@escowles
Copy link
Contributor

+1 to pcdm:hasRelatedObject, and siding with the ontology when the wiki differs.

@jpstroop
Copy link

yes, I think pcdm:hasRelatedObject is where this landed

@tpendragon
Copy link

...Really? When did that happen? At the end of the Portland talks I believe it was ore:aggregates. To reiterate, why the choice for hasRelatedObject?

@tpendragon
Copy link

So the history I can find is:

  1. Portland talks said if you have related objects, ore:aggregates. That way you can order alongside hasMember items, and both are aggregated.
  2. Hydra::PCDM work started, and the method was called #related_objects. Ticket opened at Objects can have related objects. samvera/hydra-pcdm#14 and predicate conversation ensued.
  3. Diagram updated to have hasRelatedObject, ore:aggregates was never really discussed again.
  4. Added pcdm:hasRelatedObject #6 Added to ontology.

@escowles
Copy link
Contributor

I believe the argument was this: https://github.com/duraspace/pcdm/pull/4/files#r31881404

Basically, creating our own property to define the semantics as we intend them, since the ore:aggregates description is much broader.

@tpendragon
Copy link

Alright. We'll have to update hydra-pcdm, but that's fine.

@tpendragon
Copy link

@escowles When this gets 👍'd and fixed, mind making a ticket in hydra-pcdm?

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

4 participants