NOTICE! This is a static HTML version of a legacy LOCI Software ticket.

LOCI projects are now located on GitHub. The Bio-Formats project can be found here.

Ticket #514 (closed defect: fixed)

Opened 2010-07-13T15:03:09-05:00

Last modified 2010-09-17T16:02:52-05:00

Fix Image5D and View5D view modes

Reported by: curtis Owned by: curtis
Priority: minor Milestone: bio-formats-4.2.1
Component: plugins Severity: serious
Keywords: Cc:
Blocked By: Blocking:

Description

There may be an issue with Image5D and View5D view modes, as they were not well-tested before the 4.2 release. Investigate, and make sure they work.

Relevant code is in the displayImage5D and displayView5D methods of source:trunk/components/loci-plugins/src/loci/plugins/in/DisplayHandler.java.

Attachments

image5d-z-slider.diff (163 bytes) - added by melissa 2010-07-20T09:22:53-05:00.

Change History

comment:1 Changed 2010-07-15T14:19:51-05:00 by melissa

(In [6701]) Fixed View5D instantiation; see #514.

comment:2 follow-ups: ↓ 5 ↓ 7 Changed 2010-07-15T14:27:34-05:00 by melissa

Remaining Image5D issues:

  • Z slider doesn't work (tested against andor/mark/MF-2CH-Z-T.tif). The play/pause button, however, does work.
  • An exception is thrown if the "concatenate series" option is checked (tested against andor/mark/MF-2CH-Z-T.tif). Most likely, we want to disable this option if the Image5D display mode is selected.
  • An exception is thrown when trying to open a subset of planes.

Remaining View5D issues:

  • View5D prompts for image dimensions (tested against andor/mark/VESSEL-DEC-Z.tif). I don't know what (if anything) we can do about this, but it's kind of annoying given that the original ImagePlus contains all of the dimension information.

comment:3 follow-up: ↓ 4 Changed 2010-07-15T15:23:56-05:00 by melissa

(In [6703]) Disable the "concatenate series" option when displaying with Image5D. See #514.

comment:4 in reply to: ↑ 3 Changed 2010-07-15T15:55:33-05:00 by curtis

Replying to melissa:

(In [6703]) Disable the "concatenate series" option when displaying with Image5D. See #514.

It is not clear to me why this should be necessary. The resultant ImagePlus object (after concatenation) should be no different than any other ImagePlus, so why wouldn't it be displayable within Image5D?

comment:5 in reply to: ↑ 2 Changed 2010-07-15T15:56:24-05:00 by curtis

Remaining View5D issues:

  • View5D prompts for image dimensions (tested against andor/mark/VESSEL-DEC-Z.tif). I don't know what (if anything) we can do about this, but it's kind of annoying given that the original ImagePlus contains all of the dimension information.

I thought we had found a way around this before. Perhaps a different View5D constructor? At worst, we can email the author asking for a way to avoid it.

comment:6 Changed 2010-07-16T11:30:56-05:00 by melissa

(In [6704]) * Reverted r6703 stupidity.

  • Fixed Image5D display mode to work with concatenation and range options; see #514.

comment:7 in reply to: ↑ 2 Changed 2010-07-20T09:22:14-05:00 by melissa

Replying to melissa:

Remaining Image5D issues:

  • Z slider doesn't work (tested against andor/mark/MF-2CH-Z-T.tif). The play/pause button, however, does work.

This is a bug in Image5D; the attached patch solves it.

Changed 2010-07-20T09:22:53-05:00 by melissa

comment:8 Changed 2010-08-16T15:47:48-05:00 by curtis

  • Owner changed from melissa to curtis

The remaining work for Image5D is to apply Melissa's patch to the Image5D codebase, posting the updated version to our web site, and notifying interested parties.

For View5D, we should dig a little more to eliminate the need to respecify the image dimensions, since they are already known (and I think this used to work).

comment:9 Changed 2010-09-17T15:12:50-05:00 by curtis

I have set up  Image5D in the ImageJDev.org codebase, added an  Image5D page to the web site,  committed Melissa's patch, and  released Image5D 1.2.1.

All that remains to close this ticket is to eliminate the extra View5D dialog.

comment:10 Changed 2010-09-17T16:02:52-05:00 by curtis

  • Status changed from new to closed
  • Resolution set to fixed

I reviewed the View5D code, and there is simply no way to avoid invoking a dialog pop-up when calling the View5D_ constructor with multidimensional data. So I'm going to punt on the extra View5D dialog for now—if people complain, we can contact the View5D author to resolve it.