scientist-softserv / britishlibrary

Other
3 stars 0 forks source link

(SMG) No ‘view files in viewer’ in works #521

Open NoraRamsey opened 4 months ago

NoraRamsey commented 4 months ago

No 'view files in viewer' on several works. SMG has identified over 8 works with this issue.

Image

Some examples here: Troublesome telephony: how users and non-users shaped the development of early British exchange telephony: https://sciencemuseumgroup.iro.bl.uk/concern/articles/b8626af5-5bd7-4111-a545-f73678993f8f?locale=en

The Science Museum and the Leonardo da Vinci Quincentenary Exhibition of 1952: https://sciencemuseumgroup.iro.bl.uk/concern/articles/aa569f37-36f8-4128-86b1-f7013d8e4112?locale=en

A sustainable storage solution for the Science Museum Group: https://sciencemuseumgroup.iro.bl.uk/concern/articles/73841be8-0f4d-4595-aba7-8414bf35efc5?locale=en

Through the lens of a space tourist: https://sciencemuseumgroup.iro.bl.uk/concern/articles/e1051a6c-79be-4acf-b61d-47aed60dbca4?locale=en

cziaarm commented 4 months ago

Just to update that most of these have been sorted out. However there is an issue with derivative generation for high resolution PDFs. This is rooted in a gem (mini_magick) that is used by another gem (hydra-derivatives), that is used by hyrax that is used by hyku...

6 works affected:

02c65fab-9458-41c3-9120-435348d64681 d62b24be-7832-4413-92b5-34cef573a271 87903eea-0594-404c-b4f3-718aa23ace23 e32a2c40-89b3-45ec-8bfb-095b419bad40 9c86f11b-cbb5-436a-b2cb-ee79d2dcad49 8710596f-84e2-469e-a84b-6e33302f339b

All the others should be OK now

NoraRamsey commented 4 months ago

Thanks for the update. I can see the works are all displaying correctly. The 6 works you've identified, I've reuploaded the files as pngs (ahead of SMG going live) and they are viewable now.

I can create a new ticket for high resolution pdfs so we can work on it. Am I correct in thinking it's related to what you mentioned in #510 (the http://samvera/serverless-iiif#135)

NoraRamsey commented 4 months ago

Files available in viewer on SMG tenant.