Would you please post also a win64 build with the fix? Thank you!<div>Kent</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 6, 2012 at 9:08 PM, Benjamin Gilbert <span dir="ltr"><<a href="mailto:bgilbert@cs.cmu.edu" target="_blank">bgilbert@cs.cmu.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 11/06/2012 05:55 PM, Kent Johnson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The problem is in openslide-vendor-aperio.c at line 358. When adding<br>
associated images, it assumes that if it has only seen one level, then<br>
the associated image is the thumbnail. So when there is only one level,<br>
each associated image in turn is put into the thumbnail slot.<br>
</blockquote>
<br></div>
Hi Kent,<br>
<br>
Thanks for tracking this down!<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not sure what the fix might be. Perhaps<br>
the parser in add_associated_image() could be smarter, or the default at<br>
line 358 could be taken only if there is not already an associated image.<br>
</blockquote>
<br></div>
Actually it's a pretty minor fix: we should be using the TIFF directory index to detect the thumbnail, not the number of tiled directories we've already read.<br>
<br>
Fix:<br>
<a href="https://github.com/openslide/openslide/commit/837624d7" target="_blank">https://github.com/openslide/<u></u>openslide/commit/837624d7</a><br>
<br>
Windows build with the fix:<br>
<a href="http://openslide.cs.cmu.edu/download/tmp/openslide-win32-20121106.zip" target="_blank">http://openslide.cs.cmu.edu/<u></u>download/tmp/openslide-win32-<u></u>20121106.zip</a><span class="HOEnZb"><font color="#888888"><br>
<br>
--Benjamin Gilbert<br>
<br>
</font></span></blockquote></div><br></div>