Improve UX for Bvv tracing

- Now called Stream mode
- Ensure commands relying on traditional images remain disabled/provide clearer errors
- Overall adjustments

Author: tferr

src/main/java/sc/fiji/snt/BookmarkManager.java

@@ -321,7 +321,7 @@ private void goToRow(final int tableRow) {
         } else {
             final ImagePlus imp = sntui.plugin.getImagePlus();
             if (imp == null) {
-                sntui.guiUtils.error("No image is currently open.");
+                noImageOpenError();
             } else {
                 goTo(tableRow, imp);
                 if (!sntui.plugin.getSinglePane()) {
@@ -334,6 +334,13 @@ private void goToRow(final int tableRow) {
         }
     }
 
+    private void noImageOpenError() {
+        assert sntui != null;
+        sntui.guiUtils.error((sntui.isStreamMode())
+                ? "No image is currently open. This table can be better accessed from BDV/BVV's <i>Markers</i> pane."
+                : "No image is currently open.");
+    }
+
     private void resetOrResizeColumns(final boolean reset, final boolean resize) {
         if (table == null || model == null) return;
         if (reset) GuiUtils.JTables.resetColumnOrder(table);
@@ -887,7 +894,7 @@ private void showHighlights() {
         }
         final ImagePlus imp = sntui.plugin.getImagePlus();
         if (imp == null) {
-            sntui.guiUtils.error("No image is currently open.");
+            noImageOpenError();
             syncHighlightToggle(false);
             return;
         }
@@ -1048,7 +1055,7 @@ Action goToAction() {
             @Override
             public void actionPerformed(final ActionEvent e) {
                 if (viewer == null && sntui.plugin.getImagePlus() == null) {
-                    sntui.guiUtils.error("No image is currently open.");
+                    noImageOpenError();
                     return;
                 }
                 final String pos = guiUtils.getString("Location XYZ coordinates (comma/space separated): ",

src/main/java/sc/fiji/snt/FillManagerUI.java

@@ -96,17 +96,27 @@ public enum State {READY, STARTED, ENDED, LOADED, STOPPED}
 
 
     /**
-     * Instantiates a new Fill Manager Dialog
+     * Instantiates a new Fill Manager Dialog, with no owning {@link SNTUI} (e.g. scripts/tests
+     * driving a headless-ish {@link SNT} instance that never called {@code startUI()}).
      *
-     * @param plugin the {@link SNT} instance to be associated
-     *               with this FillManager. It is assumed that its {@link SNTUI} is
-     *               available.
+     * @param plugin the {@link SNT} instance to be associated with this FillManager.
      */
     public FillManagerUI(final SNT plugin) {
-        super(plugin.getUI(), "Fill Manager");
+        this(plugin, null);
+    }
+
+    /**
+     * Instantiates a new Fill Manager Dialog owned by {@code ui}.
+     *
+     * @param plugin the {@link SNT} instance to be associated with this FillManager
+     * @param ui     the owning {@link SNTUI}, or null if none (see {@link #FillManagerUI(SNT)})
+     */
+    public FillManagerUI(final SNT plugin, final SNTUI ui) {
+        super(ui, "Fill Manager");
         getRootPane().putClientProperty("JRootPane.menuBarEmbedded", false);
 
         this.plugin = plugin;
+        final boolean streamMode = ui != null && ui.isStreamMode();
         pathAndFillManager = plugin.getPathAndFillManager();
         pathAndFillManager.addPathAndFillListener(this);
         listModel = new DefaultListModel<>();
@@ -156,6 +166,7 @@ public FillManagerUI(final SNT plugin) {
 
         // Row 1: Cursor threshold choice
         cursorThresholdChoice = new JRadioButton("Set by clicking on traced structure (preferred)");
+        cursorThresholdChoice.setEnabled(!streamMode);
         distancePanel.add(cursorThresholdChoice, gdb);
         gdb.gridy++;
 
@@ -230,7 +241,7 @@ public void focusGained(java.awt.event.FocusEvent e) {
         transparentCheckbox = new JCheckBox("Transparent overlay");
         transparentCheckbox.setToolTipText("Enabling this option allows you to better\ninspect fills, but may slow down filling");
         transparentCheckbox.addActionListener(e -> plugin.setFillTransparent(transparentCheckbox.isSelected()));
-
+        transparentCheckbox.setEnabled(!streamMode);
         final int prevLeftMargin = c.insets.left;
         c.insets.left = MARGIN;
         add(splitFillsCheckbox, c);
@@ -286,15 +297,13 @@ public void mousePressed(final MouseEvent e) {
         pack();
         adjustListPlaceholder();
         changeState(State.READY);
-        setLocationRelativeTo(plugin.getUI());
+        setLocationRelativeTo(ui);
         addWindowListener(new WindowAdapter() {
             @Override
             public void windowClosing(final WindowEvent ignored) {
                 setVisible(false);
             }
         });
-
-
     }
 
     private int[] getSelectedIndices(final String msg) {
@@ -836,8 +845,19 @@ private void assembleExportFillsMenu() {
 
         // Add image export actions
         for (ExportAction action : exportActions) {
-            JMenuItem jmi = new JMenuItem(action.getMenuText());
-            jmi.addActionListener(e -> action.execute());
+            final JMenuItem jmi = new JMenuItem(action.getMenuText());
+            jmi.addActionListener(e -> {
+                try {
+                    action.execute();
+                } catch (final UnsupportedOperationException ex) {
+                    final String msg = (plugin.getUI() != null && plugin.getUI().isStreamMode())
+                            ? "An error occurred: This operation likely requires the entire image to be loaded into memory (RAM)."
+                            : "An error occurred.";
+                    gUtils.error(msg + " See Console for details.");
+                    if (plugin.getUI() != null) plugin.getUI().setEnableDebugMode(true);
+                    SNTUtils.error(action.getMenuText(), ex);
+                }
+            });
             exportFillsMenu.add(jmi);
         }
 

src/main/java/sc/fiji/snt/PathManagerUI.java

@@ -5545,7 +5545,7 @@ private void rebuildRelationShips() {
     private boolean noValidImageDataError() {
         final boolean invalidImage = !plugin.getUI().accessToValidImagePlus();
         if (invalidImage) {
-            guiUtils.error((getSNT().getUI() != null && getSNT().getUI().getState() == SNTUI.BVV_TRACING)
+            guiUtils.error((getSNT().getUI() != null && getSNT().getUI().getState() == SNTUI.STREAMING)
                     ? "This option requires the entire image to be loaded into memory (RAM)."
                     : "This option requires valid image data to be loaded.");
         }

src/main/java/sc/fiji/snt/SNT.java

@@ -4269,7 +4269,7 @@ public void closeAndResetAllPanes() {
 		// Dispose xz/zy images unless the user stored some annotations (ROIs)
 		// on the image overlay or modified them somehow.
 		removeMIPOverlayAllPanes();
-		final boolean bvvMode = getUI() != null && getUI().getState() == SNTUI.BVV_TRACING;
+		final boolean bvvMode = getUI() != null && getUI().isStreamMode();
 		if (!single_pane) {
 			final ImagePlus[] impPanes = { xz, zy };
 			for (final ImagePlus imp : impPanes) {