-
Notifications
You must be signed in to change notification settings - Fork 24
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
Extract region and OpenSeaDragon error #138
Comments
This looks a lot like what happens when the image extract/size calculation results in a rounding error. I've patched two of these in the past and thought I had them all squashed. Does it still error out at values below 512? Can you try a few values between 512 and 1000 to see what the upper bound of the error is? What are the dimensions of your source image? |
These are the dimensions of the source image,
More information here too. Seems that regenerating the pyramid (not ideal) using vips resolves this problem, but creates another.
Now, this url (to the new tiff, not the svs) works fine:
HOWEVER, Openseadragon is not able to correctly view this image, when directed to:
Did openseadragon stop working with iiif? or is their something about this version of iiif that breaks it? |
To your other question, currently the links are live, so you can test it yourself if you want. If it is a rounding issue, it will probably require a more careful scan than I'll do by typing stuff into the search bar, so that's why I made it available (for now) at a public link. |
And here is some more information, I verified that my OSD implementation works just fine on other people's IIIF servers (not sure their implementations). Seems that the errors in OSC are tracing back to image requests it's making on serverless-iiif. For example,
returns "Not Found." These sorts of errors are not appearing when I point to example IIIF (e.g. https://iiif.princeton.edu/loris/iiif/2/pudl0001%2F4609321%2Fs42%2F00000001.jp2/info.json). So it is making me think the root cause is some thing specific with serverless-iiif. What are your thoughts? Is there a way to resolve this? |
I see a couple of issues here:
When I adjust the extract region to be within the bounds of the image, I get a valid result. After some experimentation with the first image, I've found that any request below 787 pixels wide triggers the error. I'll look at the calculator and see what I can find out. |
Looking more closely at the pyramid page dimensions:
It can't be a coincidence that the issue crops up right at one of the page boundaries, especially if regenerating the pyramid seems to resolve that part of the issue (which would rule out the rounding error I speculated about previously). Any request larger than 787 is going to use one of the larger frames and scale down, so it seems reasonable to suggest that the problem is with the data in pages 4 and 5. Regenerating the pyramid re-creates all 5 pages from the largest frame. Is it possible that the initial pyramid is corrupted or truncated, or maybe didn't convert correctly the first time? If there's a way you can share the misbehaving pyramid TIFF file ( |
I'm going to go ahead and close this since it's been a while. Feel free to reopen it or create a new issue if you still need assistance. |
I can share the misbehaving test1.svs with you here, but I'd rather not do so publicly. May I message you privately? |
Sure. My github username is also my username for gmail as well as the IIIF slack team. |
Thanks for making this software available. I'm getting an error that might trace down to node-iiif, or perhaps something wrong in my url syntax. Basically, this url in a test iiif server we set setup (using serverless-iiif 4.0.2)
https://iiif.trustedkidney.com/iiif/3/test/test1.svs/full/512,/0/default.jpg
Leads to this error:
Error: extract_area: bad extract area
But iiif certainly can read this image. These link works just fine,
It isn't clear when a size specification will fail, and it seems that it is different for each image.
What is going on here, and is there a way to fix it?
The text was updated successfully, but these errors were encountered: