Changeset 10413 for branches/Sliding Window GP/HeuristicLab.Problems.DataAnalysis.Symbolic

Timestamp:

01/28/14 19:12:39 (10 years ago)

Author:

bburlacu

Message:

#1837: Modified the SlidingWindowBestSolutionsCollection to use a dictionary for sliding window positions mapped to best solutions, because the last sliding window position before the algorithm reached its stop was not registered correctly.

Location:

branches/Sliding Window GP/HeuristicLab.Problems.DataAnalysis.Symbolic/3.4/SlidingWindow

Files:

• SlidingWindowAnalyzer.cs (modified) (2 diffs)
• SlidingWindowBestSolutionsCollection.cs (modified) (4 diffs)

branches/Sliding Window GP/HeuristicLab.Problems.DataAnalysis.Symbolic/3.4/SlidingWindow/SlidingWindowAnalyzer.cs

@@ -138 +138 @@
       }
 
+      SaveBestSolution();
+
       if (!CheckForUpdate()) return base.Apply();
-
-      SaveBestSolution();
 
       //update necessary - move sliding window
@@ -189 +189 @@
       var fitnessPartition = FitnessCalculationPartitionParameter.ActualValue;
       var best = FindBestIndividual();
-      var range = new IntRange(fitnessPartition.Start, fitnessPartition.End);
+      var range = new Tuple<int, int>(fitnessPartition.Start, fitnessPartition.End);
 
-      bestSolutionsCollection.Add(best, range);
+      bestSolutionsCollection[range] = best;
     }
 

branches/Sliding Window GP/HeuristicLab.Problems.DataAnalysis.Symbolic/3.4/SlidingWindow/SlidingWindowBestSolutionsCollection.cs

@@ -23 +23 @@
 using HeuristicLab.Common;
 using HeuristicLab.Core;
-using HeuristicLab.Data;
 using HeuristicLab.Encodings.SymbolicExpressionTreeEncoding;
 using HeuristicLab.Persistence.Default.CompositeSerializers.Storable;
+
+using SlidingWindowRange = System.Tuple<int, int>;
 
 namespace HeuristicLab.Problems.DataAnalysis.Symbolic {
@@ -31 +32 @@
   [Item("SlidingWindowBestSolutionsCollection", "An object holding a collection of the best sliding window solutions.")]
   public abstract class SlidingWindowBestSolutionsCollection : Item {
-
-    private List<ISymbolicExpressionTree> bestSolutions;
     [Storable]
-    public IEnumerable<ISymbolicExpressionTree> BestSolutions {
+    private Dictionary<SlidingWindowRange, ISymbolicExpressionTree> bestSolutions;
+    public Dictionary<SlidingWindowRange, ISymbolicExpressionTree> BestSolutions {
       get { return bestSolutions; }
-      set { bestSolutions = new List<ISymbolicExpressionTree>(value); }
-    }
-
-    private List<IntRange> slidingWindowPositions;
-    [Storable]
-    public IEnumerable<IntRange> SlidingWindowPositions {
-      get { return slidingWindowPositions; }
-      set { slidingWindowPositions = new List<IntRange>(value); }
+      set { bestSolutions = value; }
     }
 
@@ -68 +61 @@
       : base(original, cloner) {
       this.bestSolutions = original.bestSolutions;
-      this.slidingWindowPositions = original.slidingWindowPositions;
       this.problemData = original.problemData;
       this.interpreter = original.interpreter;
     }
     protected SlidingWindowBestSolutionsCollection() {
-      bestSolutions = new List<ISymbolicExpressionTree>();
-      slidingWindowPositions = new List<IntRange>();
+      bestSolutions = new Dictionary<SlidingWindowRange, ISymbolicExpressionTree>();
     }
 
-    public void Add(ISymbolicExpressionTree solution, IntRange range) {
-      if (bestSolutions == null) bestSolutions = new List<ISymbolicExpressionTree>();
-      bestSolutions.Add(solution);
-      if (slidingWindowPositions == null) slidingWindowPositions = new List<IntRange>();
-      slidingWindowPositions.Add(range);
+    public bool ContainsKey(SlidingWindowRange key) {
+      return bestSolutions.ContainsKey(key);
+    }
+
+    public ISymbolicExpressionTree this[SlidingWindowRange key] {
+      get {
+        return bestSolutions[key];
+      }
+      set {
+        if (bestSolutions.ContainsKey(key))
+          bestSolutions[key] = value;
+        else
+          bestSolutions.Add(key, value);
+      }
+    }
+
+    public void Add(SlidingWindowRange range, ISymbolicExpressionTree solution) {
+      bestSolutions.Add(range, solution);
     }
 
     public void Clear() {
       if (bestSolutions != null) bestSolutions.Clear();
-      if (slidingWindowPositions != null) slidingWindowPositions.Clear();
     }
 
@@ -92 +95 @@
      double lowerEstimationLimit = double.MinValue, double upperEstimationLimit = double.MaxValue);
 
-    public abstract ISymbolicDataAnalysisSolution CreateSolution(ISymbolicDataAnalysisModel model,
-      IDataAnalysisProblemData problemData);
+    public abstract ISymbolicDataAnalysisSolution CreateSolution(ISymbolicDataAnalysisModel model, IDataAnalysisProblemData problemData);
   }
 }