Changeset 8489 for trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/CovarianceSum.cs

Timestamp:

08/14/12 15:24:30 (12 years ago)

Author:

gkronber

Message:

#1902 removed caches in product and sum covariance functions and fixed bug in SE-ARD covariance function

File:

• trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/CovarianceSum.cs (modified) (4 diffs)

trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/CovarianceSum.cs

@@ -50 +50 @@
       this.terms = cloner.Clone(original.terms);
       this.numberOfVariables = original.numberOfVariables;
-      AttachEventHandlers();
     }
 
@@ -56 +55 @@
       : base() {
       this.terms = new ItemList<ICovarianceFunction>();
-      AttachEventHandlers();
-    }
-
-    private void AttachEventHandlers() {
-      this.terms.CollectionReset += (sender, args) => ClearCache();
-      this.terms.ItemsAdded += (sender, args) => ClearCache();
-      this.terms.ItemsRemoved += (sender, args) => ClearCache();
-      this.terms.ItemsReplaced += (sender, args) => ClearCache();
-      this.terms.ItemsMoved += (sender, args) => ClearCache();
     }
 
@@ -90 +80 @@
     }
 
-    private Dictionary<int, Tuple<int, int>> cachedParameterMap;
     public IEnumerable<double> GetGradient(double[,] x, int i, int j) {
-      //if (cachedParameterMap == null) {
-      //  CalculateParameterMap();
-      //}
-      //int ti = cachedParameterMap[k].Item1;
-      //k = cachedParameterMap[k].Item2;
-      //return terms[ti].GetGradient(x, i, j, k);
       return terms.Select(t => t.GetGradient(x, i, j)).Aggregate(Enumerable.Concat);
     }
@@ -104 +87 @@
       return terms.Select(t => t.GetCrossCovariance(x, xt, i, j)).Sum();
     }
-
-    private void ClearCache() {
-      cachedParameterMap = null;
-    }
-
-    //private void CalculateParameterMap() {
-    //  cachedParameterMap = new Dictionary<int, Tuple<int, int>>();
-    //  int k = 0;
-    //  for (int ti = 0; ti < terms.Count; ti++) {
-    //    for (int ti_k = 0; ti_k < terms[ti].GetNumberOfParameters(numberOfVariables); ti_k++) {
-    //      cachedParameterMap[k++] = Tuple.Create(ti, ti_k);
-    //    }
-    //  }
-    //}
   }
 }