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

IIIF protocol: wrong aspect ratio when requesting a region with w, as the size parameter #21

Closed
regisrob opened this issue Sep 15, 2014 · 4 comments
Assignees

Comments

@regisrob
Copy link

IIP doesn't seem to calculate the right height when requesting a region of an image with "w," in the {size} slot of the URI. Sometimes it retrieves an image with a wrong aspect ratio, and sometimes the image is corrupted too.

A few examples:
1024,1024,1024,1024/256,/0/default.jpg: outputs a 256x366 image (should be 256x256, like this)

...and another bug:
1024,1536,512,1024/256,/0/default.jpg : outputs a 256x366 image (should be 256x512)…
and when we check 1024,1536,512,1024/256,512/0/default.jpg the output image is scaled but still corrupted... unless you set an y value for region <= 1535 or >= 1537 (see 1024,1537,512,1024/256,512/0/default.jpg.
Weird.

Those bugs have been raised while testing the last version of IIP server (except latest commit a47a95f) with OpenSeadragon, which both support IIIF Image API 2.0: http://demos.biblissima-condorcet.fr/iip-osd/ (see)

@ruven: do you have an idea of what's going on? Could you confirm it's a problem on the server-side or not?

@ruven ruven self-assigned this Sep 26, 2014
@rsinghal
Copy link

I am seeing the same issue, see http://images-dev.harvardx.harvard.edu/fcgi-bin/iipsrv.fcgi?iiif=1-10_18Sept1905_tiled.tif/40960,0,6302,6814/197,/0/default.jpg, which is setting the height to 28 pixels, instead of 213.

I don't think this is an issue with my image, because when I try the same image in OSD using DeepZoom vs IIIF, the DeepZoom is crisp, while the IIIF is blurry.

@rsinghal
Copy link

rsinghal commented Nov 3, 2014

Any update?

@ruven
Copy link
Owner

ruven commented Nov 24, 2014

OK, I've just uploaded a fix for these. Please test it out and tell me if anything else comes up.

@ruven ruven closed this as completed Nov 24, 2014
@regisrob
Copy link
Author

Now it's ok, it works fine. Thanks ;-)

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