JAL-3127 drop redundant hidden representative sequences hash as parameter in ColourSc...
authorJim Procter <jprocter@issues.jalview.org>
Mon, 19 Nov 2018 17:25:42 +0000 (17:25 +0000)
committerJim Procter <jprocter@issues.jalview.org>
Mon, 19 Nov 2018 17:25:42 +0000 (17:25 +0000)
28 files changed:
src/jalview/controller/AlignViewController.java
src/jalview/gui/AnnotationColourChooser.java
src/jalview/gui/TreeCanvas.java
src/jalview/schemes/AnnotationColourGradient.java
src/jalview/schemes/Blosum62ColourScheme.java
src/jalview/schemes/BuriedColourScheme.java
src/jalview/schemes/ClustalxColourScheme.java
src/jalview/schemes/ColourSchemeI.java
src/jalview/schemes/ColourSchemes.java
src/jalview/schemes/CovariationColourScheme.java
src/jalview/schemes/FollowerColourScheme.java
src/jalview/schemes/HelixColourScheme.java
src/jalview/schemes/HydrophobicColourScheme.java
src/jalview/schemes/IdColourScheme.java
src/jalview/schemes/NucleotideColourScheme.java
src/jalview/schemes/PIDColourScheme.java
src/jalview/schemes/PurinePyrimidineColourScheme.java
src/jalview/schemes/RNAHelicesColour.java
src/jalview/schemes/RNAInteractionColourScheme.java
src/jalview/schemes/ScoreColourScheme.java
src/jalview/schemes/StrandColourScheme.java
src/jalview/schemes/TCoffeeColourScheme.java
src/jalview/schemes/TaylorColourScheme.java
src/jalview/schemes/TurnColourScheme.java
src/jalview/schemes/UserColourScheme.java
src/jalview/schemes/ZappoColourScheme.java
test/jalview/schemes/AnnotationColourGradientTest.java
test/jalview/schemes/ColourSchemesTest.java

index a5a3591..a82b98d 100644 (file)
@@ -107,8 +107,7 @@ public class AlignViewController implements AlignViewControllerI
         viewport.getAlignment().addGroup(gps[g]);
         if (colours != null)
         {
-          gps[g].setColourScheme(colours.getInstance(viewport, gps[g],
-                  viewport.getHiddenRepSequences()));
+          gps[g].setColourScheme(colours.getInstance(viewport, gps[g]));
         }
         Color col = new Color((int) (Math.random() * 255),
                 (int) (Math.random() * 255), (int) (Math.random() * 255));
index 9626309..13390da 100644 (file)
@@ -454,7 +454,7 @@ public class AnnotationColourChooser extends AnnotationRowFilter
           continue;
         }
         sg.setColourScheme(
-                acg.getInstance(av, sg, ap.av.getHiddenRepSequences()));
+                acg.getInstance(av, sg));
       }
     }
   }
index f1bdd31..f353d28 100755 (executable)
@@ -1006,8 +1006,7 @@ public class TreeCanvas extends JPanel implements MouseListener, Runnable,
         // Propagate group colours in each view
         if (viewport.getGlobalColourScheme() != null)
         {
-          cs = viewport.getGlobalColourScheme().getInstance(viewport, sg,
-                  null);
+          cs = viewport.getGlobalColourScheme().getInstance(viewport, sg);
           sg.setColourScheme(cs);
           sg.getGroupColourScheme().setThreshold(
                   viewport.getResidueShading().getThreshold(),
index 334998f..2790626 100755 (executable)
@@ -78,8 +78,7 @@ public class AnnotationColourGradient extends FollowerColourScheme
 
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+          AnnotatedCollectionI sg)
   {
     AnnotationColourGradient acg = new AnnotationColourGradient(annotation,
             getColourScheme(), aboveAnnotationThreshold);
index 202fc25..8188f4d 100755 (executable)
@@ -24,12 +24,10 @@ import jalview.analysis.scoremodels.ScoreModels;
 import jalview.api.AlignViewportI;
 import jalview.api.analysis.PairwiseScoreModelI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
 import jalview.datamodel.SequenceI;
 import jalview.util.Comparison;
 
 import java.awt.Color;
-import java.util.Map;
 
 public class Blosum62ColourScheme extends ResidueColourScheme
 {
@@ -48,8 +46,7 @@ public class Blosum62ColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new Blosum62ColourScheme();
   }
index 290ace8..e6672fc 100755 (executable)
@@ -22,11 +22,8 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -77,8 +74,7 @@ public class BuriedColourScheme extends ScoreColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new BuriedColourScheme();
   }
index 3a9f8f7..b1cfe82 100755 (executable)
@@ -337,11 +337,10 @@ public class ClustalxColourScheme extends ResidueColourScheme
 
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+          AnnotatedCollectionI sg)
   {
     ClustalxColourScheme css = new ClustalxColourScheme(sg,
-            hiddenRepSequences);
+            view.getHiddenRepSequences());
     css.includeGaps = includeGaps;
     return css;
   }
index 02ea887..7b79d88 100755 (executable)
@@ -67,12 +67,10 @@ public interface ColourSchemeI
    *          - the parent viewport
    * @param sg
    *          - the collection of sequences to be coloured
-   * @param hiddenRepSequences
    * @return copy of current scheme with any inherited settings transferred
    */
   ColourSchemeI getInstance(AlignViewportI viewport,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences);
+          AnnotatedCollectionI sg);
 
   /**
    * Answers true if the colour scheme is suitable for the given data, else
index c3d0ea0..42465f2 100644 (file)
@@ -146,7 +146,7 @@ public class ColourSchemes
     }
     ColourSchemeI cs = schemes.get(name.toLowerCase());
     return cs == null ? null
-            : cs.getInstance(viewport, forData, hiddenRepSequences);
+            : cs.getInstance(viewport, forData);
   }
 
   /**
index 41acef3..dc7971b 100644 (file)
@@ -23,8 +23,6 @@ package jalview.schemes;
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AlignmentAnnotation;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 import jalview.util.ColorUtils;
 
 import java.awt.Color;
@@ -53,8 +51,7 @@ public class CovariationColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new CovariationColourScheme(coll.getAlignmentAnnotation()[0]);
   }
index 2cfb072..24b7713 100644 (file)
@@ -22,10 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
-
-import java.util.Map;
 
 /**
  * Colourscheme that takes its colours from some other colourscheme
@@ -55,8 +51,7 @@ public class FollowerColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new FollowerColourScheme();
   }
index d260ce6..2724f77 100755 (executable)
@@ -22,11 +22,8 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 public class HelixColourScheme extends ScoreColourScheme
 {
@@ -60,8 +57,7 @@ public class HelixColourScheme extends ScoreColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new HelixColourScheme();
   }
index fcdc10f..d886bdf 100755 (executable)
@@ -22,11 +22,8 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -77,8 +74,7 @@ public class HydrophobicColourScheme extends ScoreColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new HydrophobicColourScheme();
   }
index cd26666..65dbcc9 100644 (file)
@@ -57,8 +57,7 @@ public class IdColourScheme implements ColourSchemeI
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new IdColourScheme(view, coll);
   }
index be672e3..4977107 100755 (executable)
@@ -22,10 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
-
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -61,8 +57,7 @@ public class NucleotideColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new NucleotideColourScheme();
   }
index 6275b74..3a5c066 100755 (executable)
@@ -22,13 +22,11 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
 import jalview.datamodel.SequenceGroup;
 import jalview.datamodel.SequenceI;
 import jalview.util.Comparison;
 
 import java.awt.Color;
-import java.util.Map;
 
 public class PIDColourScheme extends ResidueColourScheme
 {
@@ -94,8 +92,7 @@ public class PIDColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new PIDColourScheme();
   }
index 5c0cc1a..a8270cc 100644 (file)
@@ -22,10 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
-
-import java.util.Map;
 
 /**
  * Class is based off of NucleotideColourScheme
@@ -61,8 +57,7 @@ public class PurinePyrimidineColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new PurinePyrimidineColourScheme();
   }
index 0011d5c..33b275d 100644 (file)
@@ -208,8 +208,7 @@ public class RNAHelicesColour extends ResidueColourScheme
 
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+          AnnotatedCollectionI sg)
   {
     return new RNAHelicesColour(sg);
   }
index c2bdd96..8610417 100644 (file)
@@ -22,11 +22,9 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
 import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 public class RNAInteractionColourScheme extends ResidueColourScheme
 {
@@ -75,8 +73,7 @@ public class RNAInteractionColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new RNAInteractionColourScheme();
   }
index e0011be..eae76e1 100755 (executable)
@@ -22,12 +22,10 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
 import jalview.datamodel.SequenceI;
 import jalview.util.Comparison;
 
 import java.awt.Color;
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -121,8 +119,7 @@ public class ScoreColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new ScoreColourScheme(symbolIndex, scores, min, max);
   }
index f318b42..ef55d69 100755 (executable)
@@ -22,11 +22,8 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -77,8 +74,7 @@ public class StrandColourScheme extends ScoreColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new StrandColourScheme();
   }
index f901892..db85cc8 100644 (file)
@@ -145,8 +145,7 @@ public class TCoffeeColourScheme extends ResidueColourScheme
 
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+          AnnotatedCollectionI sg)
   {
     return new TCoffeeColourScheme(sg);
   }
index 74448e3..e2a4516 100755 (executable)
@@ -22,10 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
-
-import java.util.Map;
 
 public class TaylorColourScheme extends ResidueColourScheme
 {
@@ -52,8 +48,7 @@ public class TaylorColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new TaylorColourScheme();
   }
index 0af913b..7d8035a 100755 (executable)
@@ -22,11 +22,8 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 
 import java.awt.Color;
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -77,8 +74,7 @@ public class TurnColourScheme extends ScoreColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new TurnColourScheme();
   }
index 16c9f78..d77f2f5 100755 (executable)
@@ -22,8 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
 import jalview.util.ColorUtils;
 import jalview.util.StringUtils;
 
@@ -58,8 +56,7 @@ public class UserColourScheme extends ResidueColourScheme
 
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI sg,
-          Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+          AnnotatedCollectionI sg)
   {
     return new UserColourScheme(this);
   }
index ee3bae6..d69695f 100755 (executable)
@@ -22,10 +22,6 @@ package jalview.schemes;
 
 import jalview.api.AlignViewportI;
 import jalview.datamodel.AnnotatedCollectionI;
-import jalview.datamodel.SequenceCollectionI;
-import jalview.datamodel.SequenceI;
-
-import java.util.Map;
 
 /**
  * DOCUMENT ME!
@@ -61,8 +57,7 @@ public class ZappoColourScheme extends ResidueColourScheme
    */
   @Override
   public ColourSchemeI getInstance(AlignViewportI view,
-          AnnotatedCollectionI coll,
-          Map<SequenceI, SequenceCollectionI> hrs)
+          AnnotatedCollectionI coll)
   {
     return new ZappoColourScheme();
   }
index 4a5e057..0b0eb24 100644 (file)
@@ -184,7 +184,7 @@ public class AnnotationColourGradientTest
   {
     AnnotationColourGradient testee = new AnnotationColourGradient(ann,
             minColour, maxColour, AnnotationColourGradient.ABOVE_THRESHOLD);
-    testee = (AnnotationColourGradient) testee.getInstance(null, al, null);
+    testee = (AnnotationColourGradient) testee.getInstance(null, al);
 
     for (int col = 0; col < WIDTH; col++)
     {
@@ -226,7 +226,7 @@ public class AnnotationColourGradientTest
   {
     AnnotationColourGradient testee = new AnnotationColourGradient(ann,
             minColour, maxColour, AnnotationColourGradient.BELOW_THRESHOLD);
-    testee = (AnnotationColourGradient) testee.getInstance(null, al, null);
+    testee = (AnnotationColourGradient) testee.getInstance(null, al);
   
     for (int col = 0; col < WIDTH; col++)
     {
@@ -259,7 +259,7 @@ public class AnnotationColourGradientTest
   {
     AnnotationColourGradient testee = new AnnotationColourGradient(ann,
             minColour, maxColour, AnnotationColourGradient.NO_THRESHOLD);
-    testee = (AnnotationColourGradient) testee.getInstance(null, al, null);
+    testee = (AnnotationColourGradient) testee.getInstance(null, al);
 
     for (int col = 0; col < WIDTH; col++)
     {
@@ -278,7 +278,7 @@ public class AnnotationColourGradientTest
   {
     AnnotationColourGradient testee = new AnnotationColourGradient(ann,
             minColour, maxColour, AnnotationColourGradient.NO_THRESHOLD);
-    testee = (AnnotationColourGradient) testee.getInstance(null, al, null);
+    testee = (AnnotationColourGradient) testee.getInstance(null, al);
 
     /*
      * flag corresponding to 'use original colours' checkbox
index f6f1953..5db3743 100644 (file)
@@ -56,8 +56,7 @@ public class ColourSchemesTest
 
     @Override
     public ColourSchemeI getInstance(AlignViewportI view,
-            AnnotatedCollectionI sg,
-            Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+            AnnotatedCollectionI sg)
     {
       final ColourSchemeI cs1 = ColourSchemes.getInstance()
               .getColourScheme(JalviewColourScheme.Taylor.toString(),
@@ -147,10 +146,9 @@ public class ColourSchemesTest
 
     @Override
     public ColourSchemeI getInstance(AlignViewportI view,
-            AnnotatedCollectionI sg,
-            Map<SequenceI, SequenceCollectionI> hiddenRepSequences)
+            AnnotatedCollectionI sg)
     {
-      return new MyClustal(sg,              hiddenRepSequences);
+      return new MyClustal(sg, view.getHiddenRepSequences());
     }
 
     @Override