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

Dplamap / pcdm #5

Merged
merged 2 commits into from
Sep 22, 2015
Merged

Dplamap / pcdm #5

merged 2 commits into from
Sep 22, 2015

Conversation

azaroth42
Copy link
Contributor

closes #1

@anarchivist anarchivist self-assigned this Sep 21, 2015

#### Collections

One or more `pcdm:Collection`s map to a `dcmitypes:Collection`. As `pcdm:Collection`s may be hierarchical, and DPLA's use of `dcmitypes:Collection` is not hierarchical, information may be accumulated from multiple `pcdm:Collection`s into one.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe "Zero or more"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zero or more collections map? How would you map from no collection to a collection? ... or ... you can certainly create collections from external information, but that's not the point of this doc?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor typo: dcmitype:Collection

@anarchivist
Copy link
Member

Sidebar: We don't have to do it to get this merged, but this is probably worth at least a short conversation with @aisaac and Valentine, because it would be nice to have a similar document -- or this one -- articulate the relationship between PCDM and EDM.

@azaroth42
Copy link
Contributor Author

👍 to the suggestion. Is that a HyBox or PCDM concern, thus where to create a new issue to track it?

@anarchivist
Copy link
Member

@azaroth42 I'm inclined to suggestions there. I think it depends on how PCDM sees alignment with EDM as a priority. :)

@azaroth42
Copy link
Contributor Author

@anarchivist The same applies for IIIF, both PCDM <--> IIIF (/cc @jpstroop) and for IIIF <--> EDM.
Lets make an issue here, and reference it from other repos.

@azaroth42
Copy link
Contributor Author

Made issue #7. Anything outstanding on this PR being committed?

@anarchivist
Copy link
Member

Any interest or need in squashing the commits?

@jpstroop
Copy link

Make him squash! 😜

@no-reply
Copy link

:squash:
On Sep 21, 2015 10:52 PM, "Jon Stroop" notifications@github.com wrote:

Make him squash!


Reply to this email directly or view it on GitHub.

@azaroth42
Copy link
Contributor Author

Fiiiiine.

Sent from S6 Edge, please excuse any typos or brevity
On Sep 21, 2015 22:54, "Thomas Johnson" notifications@github.com wrote:

:squash:
On Sep 21, 2015 10:52 PM, "Jon Stroop" notifications@github.com wrote:

Make him squash!


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#5 (comment).

@azaroth42
Copy link
Contributor Author

Squashed down to initial plus revision. 👍?

anarchivist added a commit that referenced this pull request Sep 22, 2015
@anarchivist anarchivist merged commit 390ee68 into master Sep 22, 2015
@anarchivist anarchivist deleted the dplamap_pcdm branch September 22, 2015 17:10
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

Successfully merging this pull request may close these issues.

Describe relationship between DPLA MAP and PCDM
4 participants