-
Notifications
You must be signed in to change notification settings - Fork 84
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
Bundle CSV metadata into IIIF manifests #130
Comments
I tried contributing this at one point. It was decided that this is something they did not want to add. Maybe things have changed. |
That seems like good work, thanks. Reading the old discussion, it looks like most of the concerns there had to do with automatically ingesting Your code also does what I propose, which is blindly populate the 'metadata' field of the manifest from the CSV. Currently wax manifests have no metadata; https://iiif.io/api/presentation/2.1/#metadata says "A manifest SHOULD have one or more metadata pairs associated with it describing the object or work." So picking that part of the old request would bring wax into better IIIF conformance. |
+1, it seems pretty natural that Wax would at least be able to create fully fledged manifests if asked for it. If that's already possible with |
Wax cites FAIR user principles and has a re-use tab, but it shares metadata as tables rather than linking it inside the IIIF. Some forms of re-use would be better supported by including all metadata in the IIIF presentation manifests. This should be relatively simple, perhaps as a post-processing step on the current products of
wax:derivatives:iiif
if mucking around the current derivative code is too hard.It would just be adding an additional key to the json for
metadata
with the serialized row.Reference example here.
This would be a good first task for someone looking to the contribute to the codebase with little/no experience in Ruby.
The text was updated successfully, but these errors were encountered: