Changeset 13891

Timestamp:

06/11/16 13:29:45 (8 years ago)

Author:

gkronber

Message:

#2591: removed capacity initialization in CovProd and CovSum because I believe it is more costly to calculate the capacity than to just increase the list capacity when necessary.

Location:

trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/CovarianceFunctions

Files:

• CovarianceProduct.cs (modified) (1 diff)
• CovarianceSum.cs (modified) (1 diff)

trunk/sources/HeuristicLab.Algorithms.DataAnalysis/3.4/GaussianProcess/CovarianceFunctions/CovarianceProduct.cs

@@ -94 +94 @@
     public static IList<double> GetGradient(double[,] x, int i, int j, List<ParameterizedCovarianceFunction> factorFunctions) {
       var covariances = factorFunctions.Select(f => f.Covariance(x, i, j)).ToArray();
-      var gr = new List<double>(factorFunctions.Sum(f => f.CovarianceGradient(x, i, j).Count));
+      var gr = new List<double>();
       for (int ii = 0; ii < factorFunctions.Count; ii++) {
         foreach (var g in factorFunctions[ii].CovarianceGradient(x, i, j)) {

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

@@ -88 +88 @@
       sum.CrossCovariance = (x, xt, i, j) => functions.Select(e => e.CrossCovariance(x, xt, i, j)).Sum();
       sum.CovarianceGradient = (x, i, j) => {
-        var g = new List<double>(functions.Sum(e => e.CovarianceGradient(x, i, j).Count));
+        var g = new List<double>();
         foreach (var e in functions)
           g.AddRange(e.CovarianceGradient(x, i, j));