utlib / IiifItems

Embed Mirador in Omeka Classic 2.3+ for building rich IIIF-compliant exhibits
Apache License 2.0
17 stars 9 forks source link

Verifying IIIF manifests #7

Closed ChristieHurrell closed 6 years ago

ChristieHurrell commented 7 years ago

Hello, is there any documentation that goes along with this project on how to verify that the Image API for your chosen IIIF manifests is compliant with the IIIF plugin? I am working with a faculty member who is trying to import manifests from this site (http://kotenseki.nijl.ac.jp/) and although the manifests do import, we are noticing a few problems with the way they display:

To test these issues, I uploaded a manifest from a different image server, and these problems did not replicate, so I am assuming that there is an issue with the manifests referenced above. But I'm not sure how to verify that. Thanks!

dicksonlaw583 commented 7 years ago

I have taken a look at some manifests from the site, and it seems that the image API and annotation list URLs look like this:

//kotenseki.nijl.ac.jp/api/iiif/200021974/v2/NIIP/049-0081/049-0081-00003.tif //kotenseki.nijl.ac.jp/biblio/200021974/list/2

The plugin looks for http:// or https:// to determine how to start the download (which are present in all compliant manifests we have seen thus far), but the ones in the manifest don't start with either. As a result, the plugin couldn't act on these URLs, so they are left out of the import.

ChristieHurrell commented 7 years ago

Dickson, thanks for letting me know that this is the problem. Much appreciated. Would a future update to the plugin consider addressing this issue?

dicksonlaw583 commented 7 years ago

@ChristieHurrell

My apologies for the delay in responding. Can you try this branch and see if you can import? https://github.com/utlib/IiifItems/tree/patch-kotenseki

drbauder commented 6 years ago

@dicksonlaw583, I'm working with @ChristieHurrell on this project, as the server administrator. I've pulled this branch and I'm currently running the import on the manifest at http://kotenseki.nijl.ac.jp/biblio/200021974/manifest. Is this the manifest you referenced in your above comment? So far the import looks fine, but searching through the JSON manually, the URLs all appear to use https, in addition to having all forward slashes escaped (e.g., https:\/\/kotenseki.nijl.ac.jp\/api\/iiif\/200021974\/v2\/NIIP\/049-0081\/049-0081-00003.tif). Perhaps this manifest has been updated since you tested?

dicksonlaw583 commented 6 years ago

@drbauder:

Yes, the manifest has changed since the previous test, it looks standards-compliant now. The back-then-forward slashes are fine, the JSON parser will turn those back into single forward slashes.

Are the imports working properly with the unpatched plugin? If it is back in order, we can safely close this ticket.

drbauder commented 6 years ago

@dicksonlaw583,

Yep, I switched back to the main branch and the import seems to be running fine now.

Thanks for your time!

dicksonlaw583 commented 6 years ago

You're welcome. Closing.