Benjamin Gilbert
Benjamin Gilbert
I'm swamped at the moment and haven't yet had a chance to look at the patch. I haven't forgotten about it, though.
Reviewed at #157. My apologies for the extended delay. As an experiment, I used [Reviewable](https://reviewable.io/) to do the review; please use [their system](https://reviewable.io/reviews/openslide/openslide/157) for followup discussion. So far, Reviewable seems...
Could you post an example file (or send me one privately)?
Thanks for the report and sample. Fixing this will require some additional API, the details of which are still being worked out. In the short term, we could allow the...
You can pull the short-term fix from the leica-allow-multiple-main branch of bgilbert/openslide. I'm not targeting it for master for now.
Without the fix, OpenSlide refuses to open the file. VIPS then interprets it as an ordinary TIFF and gives you the first TIFF directory, which happens to be the thumbnail....
Partially fixed in master. We now render Leica regions onto a canvas the size of the entire slide. If there are multiple regions, we render them in the appropriate places....
It's a pretty major project, since multiple regions, multiple focal planes, arbitrary color channels for fluorescence imaging, and possibly high-bit-depth support are all tangled up together. (I'd rather not bolt...
Added [sample slides](http://openslide.cs.cmu.edu/download/openslide-testdata/Leica/): `Leica-2` has multiple pyramids with coplanar levels, and `Leica-3` has multiple pyramids with non-coplanar levels.
leica-allow-multiple-main wasn't (and won't be) merged to master, because it ignored extra main images without notifying the user. The partial fix mentioned in https://github.com/openslide/openslide/issues/99#issuecomment-24486004 is 8d2d98136e50f167dd76b8b8156f12498b0c6a56, which is included in...