<div dir="ltr"><div>Hi Pascal,</div><div><br></div><div>> Would it not be better to mention this in the current documentation that the</div><div>> present Bioformats Plugin is not any more supported for MIPAV?</div><div>
<br></div><div>I agree that updating the documentation is a good idea. However:</div><div><br></div><div>1) I do not control the documentation, so suggesting it to me in a private mail will not accomplish it. I have re-CCed the ome-users list.</div>
<div><br></div><div>2) Of course, I think it would be "better" to fix the plugin so that it actually works in MIPAV again.</div><div><br></div><div>Regards,<br>Curtis</div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Sat, Jun 28, 2014 at 10:03 AM, 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">
Hi Curtis,<br>
<br>
Thanks for your reply.<br>
Would it not be better to mention this in the current documentation that the present Bioformats Plugin is not any more supported for MIPAV?<br>
Right now, one gets the impression that the installation is just straight forward.<br>
<br>
I will be away till Wednesday.<br>
<br>
Best,<br>
Pascal<br>
<br>
<br>
<br>
Am 28.06.2014 um 06:51 schrieb Curtis Rueden <<a href="mailto:ctrueden@wisc.edu" target="_blank">ctrueden@wisc.edu</a>>:<br>
<div><div><br>
> Hi Pascal,<br>
><br>
> > But with respect to Zeiss czi stacks, the importer throws an error<br>
> > message that “An I/O error occurred reading the file: Import data<br>
> > error: bounds incorrect”. So I guess, as already pointed out by Roger,<br>
> > the code has some additional problems.<br>
><br>
> 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.<br>
><br>
> 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].<br>
><br>
> Regards,<br>
> Curtis<br>
><br>
> [1] <a href="https://github.com/openmicroscopy/bioformats/pull/64" target="_blank">https://github.com/openmicroscopy/bioformats/pull/64</a><br>
> [2] <a href="http://mipav.cit.nih.gov/pubwiki/index.php/MIPAV_mailing_list" target="_blank">http://mipav.cit.nih.gov/pubwiki/index.php/MIPAV_mailing_list</a><br>
><br>
><br>
> On Fri, Jun 27, 2014 at 10:14 PM, Dr. Pascal Detampel <<a href="mailto:pdetampe@ucalgary.ca" target="_blank">pdetampe@ucalgary.ca</a>> wrote:<br>
> 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" target="_blank">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" target="_blank">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>
><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>