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

567 contained examples of work #671

Merged
merged 9 commits into from
Feb 6, 2018
Merged

567 contained examples of work #671

merged 9 commits into from
Feb 6, 2018

Conversation

ChristophEwertowski
Copy link
Contributor

Resolves #567

Also #516 and lobid/lodmill#784

@ChristophEwertowski ChristophEwertowski force-pushed the 567-containedExamplesOfWork branch from 8e692cf to 3191010 Compare January 12, 2018 16:24
@ChristophEwertowski
Copy link
Contributor Author

ChristophEwertowski commented Jan 12, 2018

Will put the changes later on test. For exampleOfWork of MAB 304-1.a see CT003012479 (commit change). For 303-1 see HT018781534 (commit change).
For containsExampleOfWork (MAB 303t1) see HT018857620 (commit change).

Is it necessary to include the type Work if the properties are called "exampleOfWork" and "containsExampleOfWork"? Because there is no GND-Id for the creators I also added the living statistics at the end of the names. I didn't include MAB 521 because it has nothing to do with work titles.

Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

This already looks quite good. I commented on the examples and suggested two changes.

Is it necessary to include the type Work if the properties are called "exampleOfWork" and "containsExampleOfWork"?

We should have a type for every object if possible. So if it doesn't create any problems I suggest to keep/add it.

Because there is no GND-Id for the creators I also added the living statistics at the end of the names.

Very good. As noted in the comment on https://github.com/hbz/lobid-resources/pull/671/files#diff-9dcab9b0d92cdc0904d7aa5a68497b6d it is still missing for some.

I didn't include MAB 521 because it has nothing to do with work titles.

These might not be "work titles" in the librarian sense that they put work into making them unambiguous. However, it might make sense to use the same approach for modeling this in RDF. I am not sure about this. Let's discuss this at hbz/lobid#287 later.

"type" : [ "Work" ]
}, {
"creatorOfWork" : "Wagner, Richard 1813-1883",
"id" : "http://d-nb.info/gnd/300169914",
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks very similar to the one above but it has a different GND ID. To make the difference between those clear, we will also have to add the sub title from sub field u, here <subfield code="u">Trauermarsch</subfield>. See also #556 (comment).

@@ -30,6 +30,10 @@
}
},
"edition" : [ "Score and parts" ],
"exampleOfWork" : {
"id" : "http://d-nb.info/gnd/107360974X",
"label" : "Momento capriccioso"
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice to also add the information from subfields p and d like we do e.g. at https://github.com/hbz/lobid-resources/pull/671/files#diff-5dd561891677f7164f7d7c47a1d0438b.

@acka47 acka47 requested review from acka47 and removed request for acka47 January 15, 2018 09:54
@ChristophEwertowski
Copy link
Contributor Author

Added the improvements to test. See HT018857620 (test, json) for subfield u. There are also cases where the subfield appears multiple times.
See HT018781534 (test, json) for additional creatorOfWork and type Work.
Also added MAB 521 where no 303t exists (example HT018138676 (test, json)).
CT003012479 misses exampleOfWork (json) although it's in the branch. Have to ask @dr0i about it tomorrow.

@acka47
Copy link
Contributor

acka47 commented Jan 17, 2018

+1 from except for one last small change. I checked the GND and it uses periods as delimiter between title and subtitle(s), see e.g. http://d-nb.info/gnd/300169914. I think we should also do so.

Also, http://test.lobid.org/resources/CT003012479.json looks good for me (except for the encoding which generally is problematic on test):

{
  "id" : "http://lobid.org/resources/CT003012479#!",
  "exampleOfWork" : {
    "label" : "<<La>> Belisa Ossia La Fedeltá Riconosciuta",
    "type" : [ "Work" ]
  }
}

@ChristophEwertowski
Copy link
Contributor Author

For the new changes look at HT018857620 (json, test). For example:

  • "label" : "Sinfonien, D 759 (h-Moll)" (GND)
  • "label" : "La damnation de Faust. Invocation à la nature, Singstimme, Orchester", (GND)

@ChristophEwertowski ChristophEwertowski removed their assignment Jan 17, 2018
Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

Everything looks good now. Thanks for aligning the presentation with the GND! +1

@dr0i
Copy link
Member

dr0i commented Feb 2, 2018

+1

@dr0i dr0i added deploy and removed review labels Feb 2, 2018
@dr0i
Copy link
Member

dr0i commented Feb 5, 2018

Deployed to staging, see e.g. http://stage.lobid.org/resources/HT018857620.json.

@dr0i dr0i merged commit 00b0238 into master Feb 6, 2018
@dr0i dr0i removed the deploy label Feb 6, 2018
@dr0i
Copy link
Member

dr0i commented Feb 6, 2018

Deployed to production, see e.g. http://lobid.org/resources/HT018857620.json.
Closed.

@dr0i dr0i deleted the 567-containedExamplesOfWork branch February 6, 2018 09:19
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.

3 participants