<div dir="ltr">Hi Pascal,<div><br></div><div><div>> But with respect to Zeiss czi stacks, the importer throws an error</div><div>> message that “An I/O error occurred reading the file: Import data</div><div>> error: bounds incorrect”. So I guess, as already pointed out by Roger,</div>

<div>> the code has some additional problems.</div></div><div><br></div><div>The plan we discussed a couple of years ago with the MIPAV team was for them to adopt the plugin upstream as part of MIPAV. We did some work [1] to make it possible.</div>

<div><br></div><div>But I guess it never panned out. If you are interested in seeing that happen you could write a mail to the MIPAV mailing list [2].</div><div><br></div><div>Regards,</div><div>Curtis</div><div><br></div>

<div>[1] <a href="https://github.com/openmicroscopy/bioformats/pull/64">https://github.com/openmicroscopy/bioformats/pull/64</a></div><div>[2] <a href="http://mipav.cit.nih.gov/pubwiki/index.php/MIPAV_mailing_list">http://mipav.cit.nih.gov/pubwiki/index.php/MIPAV_mailing_list</a></div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 27, 2014 at 10:14 PM, Dr. Pascal Detampel <span dir="ltr"><<a href="mailto:pdetampe@ucalgary.ca" target="_blank">pdetampe@ucalgary.ca</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just in case somebody runs into the same issue.<br>
With the help of Roger, and some digging in the corresponding classes, I got it to work again.<br>
<br>
Next to replacing getDimensions to getPixels, dropping the index, and changing the object type to PositiveFloat (thanks Roger), I had to call the method *.getValue() instead of *.floatValue() and change the type from float to double in the lines below. Only the object "dimTimeInc” should be Double and the elements of the array “res” (a few lines below) have to be casted (float). In addition, the object type PositiveFloat has to be imported at the top of the file (import ome.xml.model.primitives.PositiveFloat;).<br>


<br>
Except for the import statement at the top, my changes looked like the following:<br>
<br>
>          // harvest physical resolution<br>
>          PositiveFloat dimPhysSizeX = store.getPixelsPhysicalSizeX(0);<br>
>          PositiveFloat dimPhysSizeY = store.getPixelsPhysicalSizeY(0);<br>
>          PositiveFloat dimPhysSizeZ = store.getPixelsPhysicalSizeZ(0);<br>
>          Double dimTimeInc = store.getPixelsTimeIncrement(0);<br>
>          double physSizeX = dimPhysSizeX == null ?<br>
>            1.0f : dimPhysSizeX.getValue();<br>
>          double physSizeY = dimPhysSizeY == null ?<br>
>            1.0f : dimPhysSizeY.getValue();<br>
>          double physSizeZ = dimPhysSizeZ == null ?<br>
>            1.0f : dimPhysSizeZ.getValue();<br>
>          float timeInc = dimTimeInc == null ? 1.0f : dimTimeInc.floatValue();<br>
><br>
>          // compute dimensional extents<br>
>          int[] dimExtents = {sizeX, sizeY, sizeZ, sizeT};<br>
>          float[] res = {(float) physSizeX, (float) physSizeY, (float) physSizeZ, timeInc};<br>
>          int[] units = {<br>
>            FileInfoBase.MICROMETERS, FileInfoBase.MICROMETERS,<br>
>            FileInfoBase.MICROMETERS, FileInfoBase.SECONDS<br>
>          };<br>
<br>
Maybe there is an more elegant way, but at least it compiles again and Mipav shows the BioFormatsImporter under Plugins.<br>
But with respect to Zeiss czi stacks, the importer throws an error message that “An I/O error occurred reading the file: Import data error: bounds incorrect”.<br>
So I guess, as already pointed out by Roger, the code has some additional problems.<br>
<br>
Thanks,<br>
Pascal<br>
<br>
<br>
On Jun 27, 2014, at 7:42 AM, OME <<a href="mailto:trac@cvs.openmicroscopy.org.uk">trac@cvs.openmicroscopy.org.uk</a>> wrote:<br>
<br>
> #12420: Bug: MIPAV importer plugin broken due to model changes<br>
> ------------------------------+---------------------------------------------<br>
>      Reporter:  rleigh       |        Owner:  mlinkert<br>
>          Type:  task         |       Status:  new<br>
>      Priority:  minor        |    Milestone:  Unscheduled<br>
>     Component:  Bio-Formats  |      Version:  5.0.2<br>
>    Resolution:               |     Keywords:<br>
> Remaining Time:               |       Sprint:<br>
>     Resources:               |   References:<br>
> Referenced By:               |<br>
> ------------------------------+---------------------------------------------<br>
> PlugInBioFormatsImporter.java will not compile:<br>
><br>
> {{{<br>
> labtop:plugins pascal$ javac -cp<br>
> /Applications/mipav:/Applications/mipav/bioformats_package.jar<br>
> PlugInBioFormatsImporter.java<br>
><br>
> PlugInBioFormatsImporter.java:201: cannot find symbol<br>
> symbol  : method getDimensionsPhysicalSizeX(int,int)<br>
> location: interface loci.formats.meta.IMetadata<br>
>           Float dimPhysSizeX = store.getDimensionsPhysicalSizeX(0, 0);<br>
><br>
> PlugInBioFormatsImporter.java:202: cannot find symbol<br>
> symbol  : method getDimensionsPhysicalSizeY(int,int)<br>
> location: interface loci.formats.meta.IMetadata<br>
>           Float dimPhysSizeY = store.getDimensionsPhysicalSizeY(0, 0);<br>
><br>
> PlugInBioFormatsImporter.java:203: cannot find symbol<br>
> symbol  : method getDimensionsPhysicalSizeZ(int,int)<br>
> location: interface loci.formats.meta.IMetadata<br>
>           Float dimPhysSizeZ = store.getDimensionsPhysicalSizeZ(0, 0);<br>
><br>
> PlugInBioFormatsImporter.java:204: cannot find symbol<br>
> symbol  : method getDimensionsTimeIncrement(int,int)<br>
> location: interface loci.formats.meta.IMetadata<br>
>           Float dimTimeInc = store.getDimensionsTimeIncrement(0, 0);<br>
><br>
> 4 errors<br>
> }}}<br>
><br>
> The method calls in question were removed from the OME data model in 2010,<br>
> and so it has been non-functional for some time.<br>
><br>
> Fixing the above error is simple (change getDimensions to getPixels and<br>
> drop the pixel index).  However, there may be additional problems with the<br>
> code.<br>
><br>
> Ideally, there should be a maven pom to allow this to be added to our<br>
> regular CI infrastructure to pick up any future breakage.<br>
><br>
> --<br>
> Ticket URL: <<a href="http://trac.openmicroscopy.org.uk/ome/ticket/12420" target="_blank">http://trac.openmicroscopy.org.uk/ome/ticket/12420</a>><br>
> OME <<a href="http://www.openmicroscopy.org/" target="_blank">http://www.openmicroscopy.org/</a>><br>
> OME Project<br>
<br>
<br>
_______________________________________________<br>
ome-users mailing list<br>
<a href="mailto:ome-users@lists.openmicroscopy.org.uk">ome-users@lists.openmicroscopy.org.uk</a><br>
<a href="http://lists.openmicroscopy.org.uk/mailman/listinfo/ome-users" target="_blank">http://lists.openmicroscopy.org.uk/mailman/listinfo/ome-users</a><br>
</blockquote></div><br></div>