X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FFeatureRenderer.java;h=c126e246d8910bc7437e62254d098076fcc30896;hb=acc9fa6c42fcdde3d55b483f1dea91e0058be06e;hp=591fa79acc6b1f05044d684a90a8608f329ccb18;hpb=50779cf7013a8db695c53637578ab135da3ce5b1;p=jalview.git

diff --git a/src/jalview/gui/FeatureRenderer.java b/src/jalview/gui/FeatureRenderer.java
index 591fa79..c126e24 100755
--- a/src/jalview/gui/FeatureRenderer.java
+++ b/src/jalview/gui/FeatureRenderer.java
@@ -41,18 +41,19 @@ public class FeatureRenderer
     FontMetrics fm;
     int charOffset;
 
+    Hashtable featureColours = new Hashtable();
+
+
     // A higher level for grouping features of a
    // particular type
     Hashtable featureGroups = null;
 
-
     // This is actually an Integer held in the hashtable,
     // Retrieved using the key feature type
     Object currentColour;
 
     String [] renderOrder;
 
-
     /**
      * Creates a new FeatureRenderer object.
      *
@@ -61,7 +62,6 @@ public class FeatureRenderer
     public FeatureRenderer(AlignViewport av)
     {
         this.av = av;
-        initColours();
     }
 
     public void transferSettings(FeatureRenderer fr)
@@ -76,33 +76,30 @@ public class FeatureRenderer
     boolean offscreenRender = false;
     public Color findFeatureColour(Color initialCol, SequenceI seq, int res)
     {
-      int seqindex = av.alignment.findIndex(seq);
-
       return new Color( findFeatureColour (initialCol.getRGB(),
-                        seqindex, res ));
+                        seq, res ));
     }
 
     /**
-     * This is used by the Molecule Viewer to get the accurate colour
-     * of the rendered sequence
+     * This is used by the Molecule Viewer and Overview to
+     * get the accurate colourof the rendered sequence
      */
-    public int findFeatureColour(int initialCol, int seqIndex, int column)
+    public int findFeatureColour(int initialCol, SequenceI seq, int column)
     {
       if(!av.showSequenceFeatures)
         return initialCol;
 
-      if(seqIndex!=lastSequenceIndex)
+      if(seq!=lastSeq)
       {
-        lastSequence = av.alignment.getSequenceAt(seqIndex);
-        lastSequenceIndex = seqIndex;
-        sequenceFeatures = lastSequence.getDatasetSequence().getSequenceFeatures();
+        lastSeq = seq;
+        sequenceFeatures = lastSeq.getDatasetSequence().getSequenceFeatures();
         if(sequenceFeatures==null)
           return initialCol;
 
         sfSize = sequenceFeatures.length;
       }
 
-      if(jalview.util.Comparison.isGap(lastSequence.getCharAt(column)))
+      if(jalview.util.Comparison.isGap(lastSeq.getCharAt(column)))
         return Color.white.getRGB();
 
 
@@ -120,7 +117,7 @@ public class FeatureRenderer
       {
         offscreenImage.setRGB(0,0,initialCol);
         drawSequence(offscreenImage.getGraphics(),
-                     lastSequence,
+                     lastSeq,
                      column,column,0);
 
         return offscreenImage.getRGB(0,0);
@@ -128,8 +125,8 @@ public class FeatureRenderer
       else
       {
         drawSequence(null,
-                    lastSequence,
-                    lastSequence.findPosition(column),
+                    lastSeq,
+                    lastSeq.findPosition(column),
                     -1, -1);
 
         if (currentColour == null)
@@ -157,8 +154,7 @@ public class FeatureRenderer
      */
    // String type;
    // SequenceFeature sf;
-    int lastSequenceIndex=-1;
-    SequenceI lastSequence;
+    SequenceI lastSeq;
     SequenceFeature [] sequenceFeatures;
     int sfSize, sfindex, spos, epos;
 
@@ -169,12 +165,13 @@ public class FeatureRenderer
           || seq.getDatasetSequence().getSequenceFeatures().length==0)
         return;
 
-
       if(g!=null)
         fm = g.getFontMetrics();
 
 
-      if (av.featuresDisplayed == null || renderOrder==null)
+      if (av.featuresDisplayed == null
+          || renderOrder==null
+          || newFeatureAdded)
        {
          findAllFeatures();
          if(av.featuresDisplayed.size()<1)
@@ -184,9 +181,9 @@ public class FeatureRenderer
          sfSize = sequenceFeatures.length;
        }
 
-       if(lastSequence==null || seq!=lastSequence)
+       if(lastSeq==null || seq!=lastSeq)
       {
-        lastSequence = seq;
+        lastSeq = seq;
         sequenceFeatures = seq.getDatasetSequence().getSequenceFeatures();
         sfSize = sequenceFeatures.length;
       }
@@ -202,8 +199,8 @@ public class FeatureRenderer
 
       if(!offscreenRender)
       {
-        spos = lastSequence.findPosition(start);
-        epos = lastSequence.findPosition(end);
+        spos = lastSeq.findPosition(start);
+        epos = lastSeq.findPosition(end);
       }
 
 
@@ -211,6 +208,7 @@ public class FeatureRenderer
       for(int renderIndex=0; renderIndex<renderOrder.length; renderIndex++)
        {
         type =  renderOrder[renderIndex];
+
         if(!av.featuresDisplayed.containsKey(type))
           continue;
 
@@ -332,9 +330,34 @@ public class FeatureRenderer
       }
     }
 
-    synchronized void findAllFeatures()
+
+    boolean newFeatureAdded = false;
+
+    public void featuresAdded()
     {
-      av.featuresDisplayed = new Hashtable();
+      findAllFeatures();
+    }
+
+   boolean findingFeatures = false;
+   synchronized void findAllFeatures()
+   {
+      newFeatureAdded = false;
+
+      if(findingFeatures)
+      {
+        newFeatureAdded = true;
+        return;
+      }
+
+      findingFeatures = true;
+      jalview.schemes.UserColourScheme ucs = new
+          jalview.schemes.UserColourScheme();
+
+      if(av.featuresDisplayed==null)
+        av.featuresDisplayed = new Hashtable();
+
+      av.featuresDisplayed.clear();
+
       Vector allfeatures = new Vector();
       for (int i = 0; i < av.alignment.getHeight(); i++)
       {
@@ -355,7 +378,11 @@ public class FeatureRenderer
               // and we don't want to render the feature in the normal way
 
               if (getColour(features[index].getType()) == null)
-                createRandomColour(features[index].getType());
+              {
+                 featureColours.put(features[index].getType(),
+                                   ucs.createColourFromName(features[index].
+                    getType()));
+              }
 
               av.featuresDisplayed.put(features[index].getType(),
                                        new Integer(getColour(features[index].
@@ -375,6 +402,8 @@ public class FeatureRenderer
         renderOrder[i] = en.nextElement().toString();
         i--;
       }
+
+      findingFeatures = false;
     }
 
     public Color getColour(String featureType)
@@ -439,59 +468,7 @@ public class FeatureRenderer
 
     }
 
-    Hashtable featureColours = new Hashtable();
-    void initColours()
-    {
-      featureColours.put("active site", new Color(255, 75, 0));
-      featureColours.put("binding site", new Color(245, 85, 0));
-      featureColours.put("calcium-binding region", new Color(235, 95, 0));
-      featureColours.put("chain", new Color(225, 105, 0));
-      featureColours.put("coiled-coil region", new Color(215, 115, 0));
-      featureColours.put("compositionally biased region", new Color(205, 125, 0));
-      featureColours.put("cross-link", new Color(195, 135, 0));
-      featureColours.put("disulfide bond", new Color(230,230,0));
-      featureColours.put("DNA-binding region", new Color(175, 155, 0));
-      featureColours.put("domain", new Color(165, 165, 0));
-      featureColours.put("glycosylation site", new Color(155, 175, 0));
-      featureColours.put("helix", new Color(145, 185, 0));
-      featureColours.put("initiator methionine", new Color(135, 195, 5));
-      featureColours.put("lipid moiety-binding region", new Color(125, 205, 15));
-      featureColours.put("metal ion-binding site", new Color(115, 215, 25));
-      featureColours.put("modified residue", new Color(105, 225, 35));
-      featureColours.put("mutagenesis site", new Color(95, 235, 45));
-      featureColours.put("non-consecutive residues", new Color(85, 245, 55));
-      featureColours.put("non-terminal residue", new Color(75, 255, 65));
-      featureColours.put("nucleotide phosphate-binding region",new Color(65, 245, 75));
-      featureColours.put("peptide", new Color(55, 235, 85));
-      featureColours.put("propeptide", new Color(45, 225, 95));
-      featureColours.put("region of interest", new Color(35, 215, 105));
-      featureColours.put("repeat", new Color(25, 205, 115));
-      featureColours.put("selenocysteine", new Color(15, 195, 125));
-      featureColours.put("sequence conflict", new Color(5, 185, 135));
-      featureColours.put("sequence variant", new Color(0, 175, 145));
-      featureColours.put("short sequence motif", new Color(0, 165, 155));
-      featureColours.put("signal peptide", new Color(0, 155, 165));
-      featureColours.put("site", new Color(0, 145, 175));
-      featureColours.put("splice variant", new Color(0, 135, 185));
-      featureColours.put("strand", new Color(0, 125, 195));
-      featureColours.put("topological domain", new Color(0, 115, 205));
-      featureColours.put("transit peptide", new Color(0, 105, 215));
-      featureColours.put("transmembrane region", new Color(0, 95, 225));
-      featureColours.put("turn", new Color(0, 85, 235));
-      featureColours.put("unsure residue", new Color(0, 75, 245));
-      featureColours.put("zinc finger region", new Color(0, 65, 255));
-    }
 
-    public Color createRandomColour(String name)
-    {
-      Random rg = new Random(System.currentTimeMillis());
-      Color color = new Color( (int) (Math.random() * 255),
-                                         (int) (Math.random() * 255),
-                                         (int) (Math.random() * 255));
-      featureColours.put(name, color); // java 1.2 here ?
-
-      return color;
-    }
 
 
 }