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

Some tiles does not fit #216

Open
Ricky-Y opened this issue Sep 3, 2021 · 4 comments
Open

Some tiles does not fit #216

Ricky-Y opened this issue Sep 3, 2021 · 4 comments

Comments

@Ricky-Y
Copy link

Ricky-Y commented Sep 3, 2021

I played around with the iipsrv and iipmooviewer on my mac, and I got them work. However, there is an issue that some tiles on bottom right corner does not fit. It happens when I use other images too. Let me paste it here:

Screen Shot 2021-09-03 at 11 51 56 AM

IIP server is version 1.1 and iipmooviewer is the current version on github. Both server and viewer is running on mac OS Catalina. The .tif file is generated by

sudo vips tiffsave <input.jpeg> NationalPark.tif --tile --pyramid --compression deflate --tile-width 256 --tile-height 256

Any thought why it could be like this?

I used identify command in terminal to test if the original image was broken but it returned good result.

@Ricky-Y
Copy link
Author

Ricky-Y commented Sep 7, 2021

BTW, I was using protocol IIIIF for the view above, when I switched to the default IIP protocol, it works perfectly

@ruven Any thought?

@ruven
Copy link
Owner

ruven commented Sep 13, 2021

It's a bug in the viewer code (not the server) - yes, use the IIP protocol for now. The server can simultaneously understand both IIP and IIIF APIs and remains IIIF compliant regardless, so it makes no difference if you use IIP with iipmooviewer.

@Ricky-Y
Copy link
Author

Ricky-Y commented Sep 14, 2021

@ruven Uh, is that bug due to the wrong size param in the tile request? The last tile should not be requested with !256,256 size since both width and height of the last tile is less than 256, is that correct?

@Ricky-Y
Copy link
Author

Ricky-Y commented Sep 14, 2021

Oh, I see the issue in viewer repo. Thanks for you reply!

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

2 participants