search

artshumrc / giza

JSON API (for TMS Database) and Django 2 application for Digital Giza
http://giza.fas.harvard.edu/
7 stars 5 forks source link

Image downloads #137

Open pmanuelian opened 3 years ago

pmanuelian commented 3 years ago

1) while we WISH all resolutions were allowed, the MFA stipulated that 1,000 pixels on the long side was their max. Assuming it's not practical to try to limit ONLY the MFA images to 1,000 pixels, do we need to blanket forbid ANY images from downloading larger that 1,000 pixels on the long side?

2) The downloaded file seems to get a random jpeg file name, not its original glass plate negative number. For example, in the attached screenshot (photo of Reisner inspecting vessel), it downloads as 460314920.jpg When its actual number is B6721_NS. I wonder if people start writing the MFA, based on this download, asking for “460314920.jpg”, the MFA won’t have any idea what they’re talking about. Can downloads retain their original filename?

Reisner
ColeDCrawford commented 3 years ago

Sometimes rendition numbers have non-English characters (German umlaut). Need to see if this will cause issues.

Either need to change the manifest to use the rendition number, or change the plugin to pull that data from the manifest.

rsinghal commented 3 years ago

Mirador is using the IIIF id from the manifest (e.g. http://giza.fas.harvard.edu/manifests/photos-96127/) sequences.canvases.images.resource.@id.

Can't change the manifest because that @id is coming from the Library, so will need to change the plugin to pull a different value from the manifest.