JAL-892 1 second delay for correct initial rendering; code tidy up
[jalview.git] / src / jalview / gui / AppVarna.java
index 9316b05..dcb2a7d 100644 (file)
  */
 package jalview.gui;
 
+import jalview.analysis.AlignSeq;
+import jalview.datamodel.AlignmentAnnotation;
+import jalview.datamodel.ColumnSelection;
+import jalview.datamodel.RnaViewerModel;
+import jalview.datamodel.SequenceGroup;
+import jalview.datamodel.SequenceI;
+import jalview.ext.varna.RnaModel;
+import jalview.structure.SecondaryStructureListener;
+import jalview.structure.SelectionListener;
+import jalview.structure.SelectionSource;
+import jalview.structure.StructureSelectionManager;
+import jalview.structure.VamsasSource;
+import jalview.util.Comparison;
+import jalview.util.MessageManager;
+import jalview.util.ShiftList;
+
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.util.Collection;
@@ -46,22 +62,6 @@ import fr.orsay.lri.varna.models.annotations.HighlightRegionAnnotation;
 import fr.orsay.lri.varna.models.rna.ModeleBase;
 import fr.orsay.lri.varna.models.rna.RNA;
 
-import jalview.analysis.AlignSeq;
-import jalview.datamodel.AlignmentAnnotation;
-import jalview.datamodel.ColumnSelection;
-import jalview.datamodel.RnaViewerModel;
-import jalview.datamodel.SequenceGroup;
-import jalview.datamodel.SequenceI;
-import jalview.ext.varna.RnaModel;
-import jalview.structure.SecondaryStructureListener;
-import jalview.structure.SelectionListener;
-import jalview.structure.SelectionSource;
-import jalview.structure.StructureSelectionManager;
-import jalview.structure.VamsasSource;
-import jalview.util.Comparison;
-import jalview.util.MessageManager;
-import jalview.util.ShiftList;
-
 public class AppVarna extends JInternalFrame implements SelectionListener,
         SecondaryStructureListener, InterfaceVARNASelectionListener,
         VamsasSource
@@ -167,8 +167,6 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
   /**
    * Constructor
    * 
-   * @param sname
-   *          a descriptive name
    * @param seq
    *          the RNA sequence
    * @param aa
@@ -176,54 +174,30 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
    * @param ap
    *          the AlignmentPanel creating this object
    */
-  public AppVarna(String sname, SequenceI seq, AlignmentAnnotation aa,
-          AlignmentPanel ap)
+  public AppVarna(SequenceI seq, AlignmentAnnotation aa, AlignmentPanel ap)
   {
     this(ap);
 
-    String name = sname + " trimmed to " + seq.getName();
-    String fullName = MessageManager.formatMessage("label.varna_params",
+    String sname = aa.sequenceRef == null ? "secondary structure (alignment)"
+            : seq.getName() + " structure";
+    String theTitle = sname
+            + (aa.sequenceRef == null ? " trimmed to " + seq.getName() : "");
+    theTitle = MessageManager.formatMessage("label.varna_params",
             new String[]
-            { name });
-    setTitle(fullName);
-
-    /*
-     * if (!aa.isValidStruc()) { throw new
-     * IllegalArgumentException("Invalid RNA structure annotation"); } final
-     * String struc = aa.getRNAStruc();
-     * 
-     * String strucseq = seq.getSequenceAsString();
-     * 
-     * String gappedTitle = sname + " (with gaps)"; String rnaTitle =
-     * gappedTitle; RNA gapped = new RNA(rnaTitle); try {
-     * gapped.setRNA(strucseq, replaceOddGaps(struc)); } catch
-     * (ExceptionUnmatchedClosingParentheses e2) { e2.printStackTrace(); } catch
-     * (ExceptionFileFormatOrSyntax e3) { e3.printStackTrace(); }
-     * models.put(gapped, new RnaModel(rnaTitle, aa, seq, gapped, true, null));
-     * 
-     * String trimmedTitle = "trimmed " + sname; rnaTitle = trimmedTitle; RNA
-     * trimmed = trimRNA(gapped, rnaTitle); models.put(trimmed, new
-     * RnaModel(rnaTitle, aa, seq, trimmed, false, null));
-     */
-    // vab = new AppVarnaBinding(Arrays.asList(new RNA[]
-    // { trimmed, gapped }));
-    // vab = new AppVarnaBinding();
-    // // String seqName = seq.getName();
-    // // String name = sname + " trimmed to " + seqName;
-    // initVarna(name);
+            { theTitle });
+    setTitle(theTitle);
 
     String gappedTitle = sname + " (with gaps)";
-    RnaModel gappedModel = new RnaModel(gappedTitle, aa, seq, null, true,
-            null);
+    RnaModel gappedModel = new RnaModel(gappedTitle, aa, seq, null, true);
     addModel(gappedModel, gappedTitle);
 
     String trimmedTitle = "trimmed " + sname;
-    RnaModel trimmedModel = new RnaModel(trimmedTitle, aa, seq, null,
-            false, null);
+    RnaModel trimmedModel = new RnaModel(trimmedTitle, aa, seq, null, false);
     addModel(trimmedModel, trimmedTitle);
     vab.setSelectedIndex(0);
   }
 
+
   /**
    * Constructor that links the viewer to a parent panel (but has no structures
    * yet - use addModel to add them)
@@ -409,25 +383,27 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
    * selected RNA in the VARNA window is highlighted at the specific position.
    * To be able to remove it before the next highlight it is saved in
    * _lastHighlight
+   * 
+   * @param sequence
+   * @param index
+   *          the aligned sequence position (base 0)
+   * @param position
+   *          the dataset sequence position (base 1)
    */
   @Override
-  public void mouseOverSequence(SequenceI sequence, int index)
+  public void mouseOverSequence(SequenceI sequence, final int index,
+          final int position)
   {
     RNA rna = vab.getSelectedRNA();
     if (rna == null)
     {
       return;
     }
-    if (models.get(rna).seq == sequence)
+    RnaModel rnaModel = models.get(rna);
+    if (rnaModel.seq == sequence)
     {
-      ShiftList shift = offsets.get(rna);
-      if (shift != null)
-      {
-        // System.err.print("Orig pos:"+index);
-        index = shift.shift(index);
-        // System.err.println("\nFinal pos:"+index);
-      }
-      mouseOverHighlighter.highlightRegion(rna, index, index);
+      int highlightPos = rnaModel.gapped ? index : position - 1;
+      mouseOverHighlighter.highlightRegion(rna, highlightPos, highlightPos);
       vab.updateSelectedRNA(rna);
     }
   }
@@ -614,35 +590,6 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
     }
 
     /*
-     * loaded from project file with Varna session data
-     */
-    if (model.varnaSession != null)
-    {
-      try
-      {
-        FullBackup fromSession = vab.vp.loadSession(model.varnaSession);
-        vab.addStructure(fromSession.rna, fromSession.config);
-        RNA rna = fromSession.rna;
-        // copy the model, but now including the RNA object
-        RnaModel newModel = new RnaModel(model.title, model.ann, model.seq,
-                rna, model.gapped, model.varnaSession);
-        if (!model.gapped)
-        {
-          registerOffset(rna, buildOffset(model.seq));
-        }
-        // TODO and add mapping (offsets)
-        models.put(rna, newModel);
-        // capture rna selection state when saved
-        selectionHighlighter = new VarnaHighlighter(rna);
-        return fromSession.rna;
-      } catch (ExceptionLoadingFailed e)
-      {
-        e.printStackTrace();
-        return null;
-      }
-    }
-
-    /*
      * opened on request in Jalview session
      */
     RNA rna = new RNA(modelName);
@@ -666,7 +613,7 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
       rna = trimRNA(rna, modelName);
     }
     models.put(rna, new RnaModel(modelName, model.ann, model.seq, rna,
-            model.gapped, null));
+            model.gapped));
     vab.addStructure(rna);
     return rna;
   }
@@ -680,6 +627,7 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
   protected ShiftList buildOffset(SequenceI seq)
   {
     // TODO refactor to avoid duplication with trimRNA()
+    // TODO JAL-1789 bugs in use of ShiftList here
     ShiftList offset = new ShiftList();
     int ofstart = -1;
     int sleng = seq.getLength();
@@ -715,10 +663,64 @@ public class AppVarna extends JInternalFrame implements SelectionListener,
   /**
    * Set the selected index in the model selection list
    * 
-   * @param selectedRna
+   * @param selectedIndex
    */
-  public void setSelectedIndex(int selectedRna)
+  public void setInitialSelection(final int selectedIndex)
   {
-    vab.setSelectedIndex(selectedRna);
+    /*
+     * empirically it needs a second for Varna/AWT to finish loading/drawing
+     * models for this to work; SwingUtilities.invokeLater _not_ a solution;
+     * explanation and/or better solution welcome!
+     */
+    synchronized (this)
+    {
+      try
+      {
+        wait(1000);
+      } catch (InterruptedException e)
+      {
+        // meh
+      }
+    }
+    vab.setSelectedIndex(selectedIndex);
+  }
+
+
+  /**
+   * Add a model with associated Varna session file
+   * 
+   * @param rna
+   * @param modelName
+   */
+  public RNA addModelSession(RnaModel model, String modelName,
+          String sessionFile)
+  {
+    if (!model.ann.isValidStruc())
+    {
+      throw new IllegalArgumentException("Invalid RNA structure annotation");
+    }
+
+    try
+    {
+      FullBackup fromSession = vab.vp.loadSession(sessionFile);
+      vab.addStructure(fromSession.rna, fromSession.config);
+      RNA rna = fromSession.rna;
+      // copy the model, but now including the RNA object
+      RnaModel newModel = new RnaModel(model.title, model.ann, model.seq,
+              rna, model.gapped);
+      if (!model.gapped)
+      {
+        registerOffset(rna, buildOffset(model.seq));
+      }
+      models.put(rna, newModel);
+      // capture rna selection state when saved
+      selectionHighlighter = new VarnaHighlighter(rna);
+      return fromSession.rna;
+    } catch (ExceptionLoadingFailed e)
+    {
+      System.err
+              .println("Error restoring Varna session: " + e.getMessage());
+      return null;
+    }
   }
 }