JAL-3253 temporary branch SwingJS upgrade with testNG fixes Java 8
[jalview.git] / src / jalview / datamodel / features / SequenceFeatures.java
index 0d29184..2101a2f 100644 (file)
@@ -23,9 +23,11 @@ package jalview.datamodel.features;
 import jalview.datamodel.SequenceFeature;
 import jalview.io.gff.SequenceOntologyFactory;
 import jalview.io.gff.SequenceOntologyI;
+import jalview.util.Platform;
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
@@ -45,7 +47,6 @@ import intervalstore.api.IntervalI;
  */
 public class SequenceFeatures implements SequenceFeaturesI
 {
-
   /*
    * map from feature type to structured store of features for that type
    * null types are permitted (but not a good idea!)
@@ -111,7 +112,7 @@ public class SequenceFeatures implements SequenceFeaturesI
     List<SequenceFeature> result = new ArrayList<>();
     for (FeatureStore featureSet : varargToTypes(type))
     {
-      result.addAll(featureSet.findOverlappingFeatures(from, to));
+      featureSet.findFeatures(from, to, result);
     }
     return result;
   }
@@ -151,8 +152,8 @@ public class SequenceFeatures implements SequenceFeaturesI
       return new ArrayList<>();
     }
 
-    return getAllFeatures(featureTypes.toArray(new String[featureTypes
-            .size()]));
+    return getAllFeatures(
+            featureTypes.toArray(new String[featureTypes.size()]));
   }
 
   /**
@@ -195,7 +196,7 @@ public class SequenceFeatures implements SequenceFeaturesI
 
     for (FeatureStore featureSet : varargToTypes(type))
     {
-      result.addAll(featureSet.getPositionalFeatures());
+      featureSet.getPositionalFeatures(result);
     }
     return result;
   }
@@ -217,7 +218,6 @@ public class SequenceFeatures implements SequenceFeaturesI
       return featureStore.values();
     }
 
-
     List<FeatureStore> types = new ArrayList<>();
     List<String> args = Arrays.asList(type);
     for (Entry<String, FeatureStore> featureType : featureStore.entrySet())
@@ -240,7 +240,7 @@ public class SequenceFeatures implements SequenceFeaturesI
 
     for (FeatureStore featureSet : varargToTypes(type))
     {
-      result.addAll(featureSet.getContactFeatures());
+      featureSet.getContactFeatures(result);
     }
     return result;
   }
@@ -255,7 +255,7 @@ public class SequenceFeatures implements SequenceFeaturesI
 
     for (FeatureStore featureSet : varargToTypes(type))
     {
-      result.addAll(featureSet.getNonPositionalFeatures());
+      featureSet.getNonPositionalFeatures(result);
     }
     return result;
   }
@@ -299,7 +299,9 @@ public class SequenceFeatures implements SequenceFeaturesI
   public Set<String> getFeatureGroups(boolean positionalFeatures,
           String... type)
   {
-    Set<String> groups = new HashSet<>();
+    // BH 2020.03.21 This is the set that orders the list of groups
+    // at the top of the FeatureSettings panel.
+    Set<String> groups = Platform.getJavaOrderedHashSet();
 
     for (FeatureStore featureSet : varargToTypes(type))
     {
@@ -316,12 +318,15 @@ public class SequenceFeatures implements SequenceFeaturesI
   public Set<String> getFeatureTypesForGroups(boolean positionalFeatures,
           String... groups)
   {
-    Set<String> result = new HashSet<>();
+    // BH 2020.03.21 This set is the one that sets the initial ordering for
+    // feature rendering. We set it to new HashSet<>(16,0.75) to force it to
+    // be backed by a Java hash-ordered HashMap instead of a JavaScript Map.
+    Set<String> result = Platform.getJavaOrderedHashSet();
 
     for (Entry<String, FeatureStore> featureType : featureStore.entrySet())
     {
-      Set<String> featureGroups = featureType.getValue().getFeatureGroups(
-              positionalFeatures);
+      Set<String> featureGroups = featureType.getValue()
+              .getFeatureGroups(positionalFeatures);
       for (String group : groups)
       {
         if (featureGroups.contains(group))
@@ -344,7 +349,7 @@ public class SequenceFeatures implements SequenceFeaturesI
   @Override
   public Set<String> getFeatureTypes(String... soTerm)
   {
-    Set<String> types = new HashSet<>();
+    Set<String> types = new HashSet<>(15, 0.75f);
     for (Entry<String, FeatureStore> entry : featureStore.entrySet())
     {
       String type = entry.getKey();
@@ -389,8 +394,9 @@ public class SequenceFeatures implements SequenceFeaturesI
   @Override
   public float getMinimumScore(String type, boolean positional)
   {
-    return featureStore.containsKey(type) ? featureStore.get(type)
-            .getMinimumScore(positional) : Float.NaN;
+    return featureStore.containsKey(type)
+            ? featureStore.get(type).getMinimumScore(positional)
+            : Float.NaN;
   }
 
   /**
@@ -399,8 +405,9 @@ public class SequenceFeatures implements SequenceFeaturesI
   @Override
   public float getMaximumScore(String type, boolean positional)
   {
-    return featureStore.containsKey(type) ? featureStore.get(type)
-            .getMaximumScore(positional) : Float.NaN;
+    return featureStore.containsKey(type)
+            ? featureStore.get(type).getMaximumScore(positional)
+            : Float.NaN;
   }
 
   /**
@@ -413,7 +420,9 @@ public class SequenceFeatures implements SequenceFeaturesI
   public static void sortFeatures(List<? extends IntervalI> features,
           final boolean forwardStrand)
   {
-    IntervalI.sortIntervals(features, forwardStrand);
+    Collections.sort(features,
+            forwardStrand ? IntervalI.COMPARE_BEGIN_ASC
+                    : IntervalI.COMPARE_END_DESC);
   }
 
   /**
@@ -465,69 +474,23 @@ public class SequenceFeatures implements SequenceFeaturesI
     featureStore.clear();
   }
 
-  /**
-   * Simplified find for features associated with a given position.
-   * 
-   * JavaScript set to not use IntervalI, but easily testable by setting false
-   * to true in javadoc
-   * 
-   * FeatureRenderer has checked already that featureStore does contain type.
-   * 
-   * @author Bob Hanson 2019.07.30
-   */
   @Override
   public List<SequenceFeature> findFeatures(int pos, String type,
           List<SequenceFeature> list)
   {
     FeatureStore fs = featureStore.get(type);
-    boolean useIntervalStore = /**
-                                * @j2sNative false &&
-                                */
-            true;
-    return (useIntervalStore ? fs.findOverlappingFeatures(pos, pos)
-            : fs.findOverlappingFeatures(pos, list));
+    if (fs == null)
+    {
+      return list == null ? new ArrayList<>() : list;
+    }
+    return fs.findFeatures(pos, pos, list);
   }
 
-  // Chrome; developer console closed
-
-  // BH 2019.08.01 useIntervalStore true, redraw false:
-  // Platform: timer mark 13.848 0.367 overviewrender 16000 pixels row:14
-  // Platform: timer mark 15.391 0.39 overviewrender 16000 pixels row:14
-  // Platform: timer mark 16.498 0.39 overviewrender 16000 pixels row:14
-  // Platform: timer mark 17.596 0.401 overviewrender 16000 pixels row:14
-  // Platform: timer mark 18.738 0.363 overviewrender 16000 pixels row:14
-  // Platform: timer mark 19.659 0.358 overviewrender 16000 pixels row:14
-  // Platform: timer mark 20.737 0.359 overviewrender 16000 pixels row:14
-  // Platform: timer mark 21.797 0.391 overviewrender 16000 pixels row:14
-  // Platform: timer mark 22.851 0.361 overviewrender 16000 pixels row:14
-  // Platform: timer mark 24.019 0.395 overviewrender 16000 pixels row:14
-
-  // BH 2019.08.01 useIntervalStore false, redraw false:
-  // Platform: timer mark 19.011 0.181 overviewrender 16000 pixels row:14
-  // Platform: timer mark 20.311 0.183 overviewrender 16000 pixels row:14
-  // Platform: timer mark 21.368 0.175 overviewrender 16000 pixels row:14
-  // Platform: timer mark 22.347 0.178 overviewrender 16000 pixels row:14
-  // Platform: timer mark 23.605 0.216 overviewrender 16000 pixels row:14
-  // Platform: timer mark 24.836 0.191 overviewrender 16000 pixels row:14
-  // Platform: timer mark 26.016 0.181 overviewrender 16000 pixels row:14
-  // Platform: timer mark 27.278 0.178 overviewrender 16000 pixels row:14
-  // Platform: timer mark 28.158 0.181 overviewrender 16000 pixels row:14
-  // Platform: timer mark 29.227 0.196 overviewrender 16000 pixels row:14
-  // Platform: timer mark 30.1 0.171 overviewrender 16000 pixels row:14
-  // Platform: timer mark 31.684 0.196 overviewrender 16000 pixels row:14
-  // Platform: timer mark 32.779 0.18 overviewrender 16000 pixels row:14
-  // Platform: timer mark 52.355 0.185 overviewrender 16000 pixels row:14
-  // Platform: timer mark 53.829 0.186 overviewrender 16000 pixels row:14
-
-
-
-  /**
-   * @author Bob Hanson 2019.08.01
-   */
   @Override
   public boolean hasFeatures(String type)
   {
-    return featureStore.containsKey(type);
+    return featureStore.containsKey(type)
+            && !featureStore.get(type).isEmpty();
   }
 
 }