NOTICE! This is a static HTML version of a legacy Fiji BugZilla bug.

The Fiji project now uses GitHub Issues for issue tracking.

Please file all new issues there.

Bug 29 - Some plugins freeze Fiji
Some plugins freeze Fiji
Status: RESOLVED LATER
Product: Fiji
Classification: Unclassified
Component: Plugins
unspecified
Macintosh Mac OS
: P2 normal
Assigned To: Mark Longair
Depends on:
Blocks:
 
Reported: 2009-01-28 19:55 CST by Jean-Yves Tinevez
Modified: 2012-01-12 09:44 CST
0 users

See Also:


Attachments
thread states on plugin hang (7.36 KB, application/octet-stream)
2009-09-30 19:41 CDT, Mark Longair

Description Jean-Yves Tinevez 2009-01-28 19:55:06 CST
Two plugins of the submenu Process make Fiji freeze (on have to "force quit" it afterwards):
- Shape-based averaging
- Quantile Based Normalization

To reproduce, just launch them.
Comment 1 Johannes Schindelin 2009-02-13 15:31:18 CST
This seems to be triggered by having a modal (i.e. blocking input to other windows) dialog launch another modal dialog.
Comment 2 Mark Longair 2009-02-16 13:16:42 CST
It sounds as if Johannes is already on the case, but I'm afraid I can't reproduce this on my Mac.  What versions of Mac OS and Java is this seen on?
Comment 3 Jean-Yves Tinevez 2009-02-17 10:17:08 CST
(In reply to comment #2)
> It sounds as if Johannes is already on the case, but I'm afraid I can't
> reproduce this on my Mac.  What versions of Mac OS and Java is this seen on?
> 
I could not reproduce it for Shape-based averaging. But it still fails for Quantile Based normalization (v1.v).
Here are the details of the machine incriminated:
MacPro, running MacOSX 10.5.6.
Java is:
Java properties applets can read:
  java.version: 1.5.0_16
  java.vendor: Apple Inc.
  mrj.version: 1050.1.5.0_16-284
  os.name: Mac OS X
  os.version: 10.5.6
  os.arch: x86_64
  file.separator: /
  path.separator: :
  line.separator: <lf>

Comment 4 Mark Longair 2009-09-29 20:26:09 CDT
I tried to find a Mac with Mac OS later than 10.4 today, but apparently I don't have a login on any computer with 10.5 or later in the lab.  However, just in case the fix is as simple as making the original dialogs non-modal, as Johannes suggests might be the case, I've pushed a branch modeless-fix-for-mac-os of VIB:

  http://pacific.mpi-cbg.de/cgi-bin/gitweb.cgi?p=VIB.git;a=commit;h=cfffe3433011c445e4aa6c73342233e46b08d60d;hb=refs/heads/VIB

Jean-Yves, could you try that out and see if it fixes the bug?

many thanks,
mark
Comment 5 Jean-Yves Tinevez 2009-09-30 11:20:37 CDT
> http://pacific.mpi-cbg.de/cgi-bin/gitweb.cgi?p=VIB.git;a=commit;h=cfffe3433011c445e4aa6c73342233e46b08d60d;hb=refs/heads/VIB
> 
> Jean-Yves, could you try that out and see if it fixes the bug?
> 
Hey
Unfortunately, the symptoms stay the same. 
Can someone confirm?
Comment 6 Mark Longair 2009-09-30 12:44:48 CDT
Oh well.  I'll have access to an appropriate Mac late this afternoon, and I'll have another go then.
Comment 7 Mark Longair 2009-09-30 19:41:34 CDT
Created attachment 6
thread states on plugin hang
Comment 8 Mark Longair 2009-09-30 19:47:31 CDT
I don't properly understand this yet, but here's some more information.  With VIB at d23b0e8d5775602c92 (i.e. using GenericDialog as before) the plugin seems to be waiting forever on AWTTreeLock.  (I've attached the kill -3 dump of the thread states.)  I think this is something to do with the ScrollPane used in vib.app.gui.FileGroupDialog, since with the following patch (just adding the List directly, rather than in a ScrollPane) it doesn't happen:

iff --git a/vib/app/gui/FileGroupDialog.java b/vib/app/gui/FileGroupDialog.java
index 47f00fe..dc75049 100644
--- a/vib/app/gui/FileGroupDialog.java
+++ b/vib/app/gui/FileGroupDialog.java
@@ -76,15 +76,12 @@ public class FileGroupDialog extends Panel
                gridbag.setConstraints(name, c);
                this.add(name);
 
-               ScrollPane scroll = new ScrollPane();
-               scroll.add(list);
-               scroll.setPreferredSize(new Dimension(300,100));
                c.gridx = 0;
                c.gridy++;
                c.fill = GridBagConstraints.BOTH;
                c.weightx = c.weighty = 0.5;
-               gridbag.setConstraints(scroll, c);
-               this.add(scroll);
+               gridbag.setConstraints(list, c);
+               this.add(list);
 
                Panel buttons = new Panel(new GridLayout(3,1));
                add = new Button("Add to files");
Comment 9 Jean-Yves Tinevez 2009-10-02 13:17:16 CDT
(In reply to comment #8)
> I don't properly understand this yet, but here's some more information.  With
> VIB at d23b0e8d5775602c92 (i.e. using GenericDialog as before) the plugin seems
> to be waiting forever on AWTTreeLock.  (I've attached the kill -3 dump of the
> thread states.)  I think this is something to do with the ScrollPane used in
> vib.app.gui.FileGroupDialog, since with the following patch (just adding the
> List directly, rather than in a ScrollPane) it doesn't happen:
> 

I could not apply the patch. 
Can you push it to the modeless branch as before?
Comment 10 Mark Longair 2009-10-02 14:12:33 CDT
(In reply to comment #9)
> (In reply to comment #8)
> > I don't properly understand this yet, but here's some more information.  With
> > VIB at d23b0e8d5775602c92 (i.e. using GenericDialog as before) the plugin seems
> > to be waiting forever on AWTTreeLock.  (I've attached the kill -3 dump of the
> > thread states.)  I think this is something to do with the ScrollPane used in
> > vib.app.gui.FileGroupDialog, since with the following patch (just adding the
> > List directly, rather than in a ScrollPane) it doesn't happen:
> > 
> 
> I could not apply the patch. 
> Can you push it to the modeless branch as before?
> 

I don't think the modeless branch is relevant any more, so I've pushed it to a new branch, called "scrollpane-non-fix":

  http://pacific.mpi-cbg.de/cgi-bin/gitweb.cgi?p=VIB.git;a=commitdiff;h=6b5139ee43075844072aaffaa99766efc9a5d67b
Comment 11 Jean-Yves Tinevez 2009-10-02 14:51:12 CDT
Ok this fix works.
The plugin does not crash Fiji, but I could not test it, because I do not know what it does :)
Comment 12 Mark Longair 2009-10-02 15:23:54 CDT
Note that it's not really a fix, since the dialog is unusable for more than a few images without the scroll bars.

I have more urgent things to deal with, unfortunately, so if someone had time (and an available Mac OS X 10.5 machine( in order to investigate why there's the hang in first place that would be great.  Otherwise I'll just disable them on Mac OS 10.5 (with a warning dialog explaining) for this release.
Comment 13 Jean-Yves Tinevez 2009-10-02 15:43:50 CDT
We can remove them from staged-plugin, and see if the updated will be able to put them back when they are ready.
They are anyway not documented.
Comment 14 Mark Longair 2009-10-02 16:44:51 CDT
(In reply to comment #13)
> We can remove them from staged-plugin, and see if the updated will be able to
> put them back when they are ready.
> They are anyway not documented.

Shape_Based_Averaging should be removed for this release, but for different reasons, as discussed in IRC.

Quantile_Based_Normalization *is* documented, however:

  http://homepages.inf.ed.ac.uk/s9808248/imagej/quantile-normalization/

... which has been pointed to by this page for some time.

  http://pacific.mpi-cbg.de/wiki/index.php/Quantile_Based_Normalization

Please don't just remove plugins that people use because they don't work on particular versions of Mac OS.  What I suggested in comment 12 will be fine until I have more time to look at this.  (That'll certainly be after I've come back from holiday, on the 12th of October.)
Comment 15 Jean-Yves Tinevez 2009-10-02 16:47:45 CDT
Oops sorry. Then i did not get what you meant by disabling...
Comment 16 Mark Longair 2009-10-03 00:28:33 CDT
I've just pushed something that does that selective disabling to the VIB branch of VIB:

  http://pacific.mpi-cbg.de/cgi-bin/gitweb.cgi?p=VIB.git;a=commitdiff;h=c8982b4d4c7fad8b9fc85bc0b278396f30cfd601

I'll look at this more when I'm back, and hopefully feeling better.
Comment 17 Jean-Yves Tinevez 2009-10-15 16:16:20 CDT
(In reply to comment #16)
> I've just pushed something that does that selective disabling to the VIB branch

The selective disabling work. It is not in new release tough.
Comment 18 Jan Eglinger 2012-01-11 15:21:08 CST
I just stumbled upon this bug because I tried to run "Quantile Based Normalization" on a Mac, and it is disabled.
Since the report has been a while ago, I wondered if the cause of this bug maybe disappeared in the meantime.

Any new information?
Comment 19 Mark Longair 2012-01-12 09:44:59 CST
Hi Jan,

I'm travelling today, but I'll try to get access to a Mac tomorrow to see whether this bug still exists.

regards,
mark