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

Image downloads #137

Open
pmanuelian opened this issue Feb 18, 2021 · 2 comments
Open

Image downloads #137

pmanuelian opened this issue Feb 18, 2021 · 2 comments
Assignees

Comments

@pmanuelian
Copy link

  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
Copy link
Contributor

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
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants