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

Support choice of images/videos in a single canvas #108

Closed
masaball opened this issue Feb 9, 2023 · 3 comments
Closed

Support choice of images/videos in a single canvas #108

masaball opened this issue Feb 9, 2023 · 3 comments

Comments

@masaball
Copy link

masaball commented Feb 9, 2023

Avalon Media System manifests use the Choice construct described in Cookbook recipe 33 to provide users with quality options. Currently this is an unsupported feature in Clover that prevents Avalon manifests from loading properly:
Screenshot_20230209_094036

Clover currently requests the manifest and loads the object metadata but fails to parse the Body to request the actual .m3u8 file(s).

Even if there is no desire to provide full support for this, would it be possible to implement minimum handling to parse the Choice construct and request the first item in the list?

@hadro
Copy link

hadro commented Jun 26, 2023

A +1 to this issue -- as we're working with the Internet Archive to update their IIIF infrastructure, including support for millions of A/V items, we're currently looking at using choice to handle the wide variety of files and derivatives that IA makes available for every A/V item.

However, this means that most items don't seem to render when I try them using Clover.

Here's an example manifest that doesn't seem to work in clover for the reasons above (with the caveat that this is currently just a test URL and some elements are liable to change):
https://archivelabs-iiif-archivelab-org-presentation-v3.ux-fnf-misc.archive.org/iiif/3/HHFBC_tapes_D54A/manifest.json

@hadro
Copy link

hadro commented Sep 14, 2023

FWIW, here's the production version of the manifest in the comment above:
https://iiif.archive.org/iiif/HHFBC_tapes_D54A/manifest.json
And this is what currently renders in the Clover interface

@mathewjordan
Copy link
Member

@masaball cleaning up some old issues here and closing this. This is now supported. Thank you for opening the issue!

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

No branches or pull requests

3 participants