-
Notifications
You must be signed in to change notification settings - Fork 5
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
Issue with multiple page item from IIIF training #46
Comments
This is caused by the same issue we were discussing on Slack with Sara, our code expects type Should be an easy enough fix, we can just look to see how many image files with |
Although looking at the item in question, I think the mediatype might be wrong, as in addition to the static images in the top level directory, there are also zipfiles with JP2s that look as if they have been processed by the BookReader code, but because the mediatype is not |
The images for these zipped items are available via Cantaloupe: https://iiif.archive.org/image/iiif/3/st-anthony-relics-01%2fASB-Consorzio_jp2.zip%2fASB-Consorzio_jp2%2FASB-Consorzio_0000.jp2/full/max/0/default.jpg |
The V2 manifest issue may be related - the code downloads the item in order to open it with PIL to get dimension data etc, but it is quite naive and it ends up downloading the first
This is the file that it downloads:
And the matching type and bytesize from the metadata:
So I think I would say that we should deal with the multiple images in an |
Duplicate of #52 |
Re-target as a cantaloupe issue. |
This is a very complicated example of: It contains a number of PDF documents and jpg images. Currently only the PDFs are shown in the Internet Archive viewer. |
This came up in the IIIF training:
https://archive.org/details/st-anthony-relics-01/
It contains 5 images but the v3 manifets contains 1 image:
https://iiif.archive.org/iiif/3/st-anthony-relics-01/manifest.json
and the v2 manifest doesn't work:
https://iiif.archive.org/iiif/2/st-anthony-relics-01/manifest.json
The text was updated successfully, but these errors were encountered: