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

Tile width/length must be a multiple of 16 #88

Closed
gsakkis opened this issue Nov 13, 2022 · 2 comments · Fixed by #89
Closed

Tile width/length must be a multiple of 16 #88

gsakkis opened this issue Nov 13, 2022 · 2 comments · Fixed by #89

Comments

@gsakkis
Copy link

gsakkis commented Nov 13, 2022

After converting CMU-1-Small-Region.svs to OME-Zarr via bioformats2raw and then converting the latter to OME-TIFF by raw2ometiff, the tile width and/or length of some pages is not a multiple of 16:

$ tiffinfo CMU-1-Small-Region.ome.tiff > /dev/null 
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile width 387, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 463, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile width 387, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 463, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile width 387, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 463, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 431, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 431, convert file.
CMU-1-Small-Region.ome.tiff: Warning, Nonstandard tile length 431, convert file.
@gsakkis gsakkis changed the title TileWidth and TileLength must be a multiple of 16 Tile width/length must be a multiple of 16 Nov 13, 2022
melissalinkert added a commit to melissalinkert/raw2ometiff-1 that referenced this issue Nov 14, 2022
@melissalinkert
Copy link
Member

Thanks, @gsakkis. #89 is a proposed fix; if you have a chance to try that out, we'd welcome your feedback.

@gsakkis
Copy link
Author

gsakkis commented Dec 8, 2022

@melissalinkert I tested the 0.4.0 release and the issue has indeed been resolved; thanks and apologies for the delayed 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

Successfully merging a pull request may close this issue.

2 participants