Changeset 13900

Timestamp:

06/16/16 16:42:37 (6 years ago)

Author:

mkommend

Message:

#2609: Refactored multiplication and division handling during transformationo to AutoDiff terms in the CoOp-Evaluator.

File:

• trunk/sources/HeuristicLab.Problems.DataAnalysis.Symbolic.Regression/3.4/SingleObjective/Evaluators/SymbolicRegressionConstantOptimizationEvaluator.cs (modified) (1 diff)

trunk/sources/HeuristicLab.Problems.DataAnalysis.Symbolic.Regression/3.4/SingleObjective/Evaluators/SymbolicRegressionConstantOptimizationEvaluator.cs

@@ -336 +336 @@
       }
       if (node.Symbol is Multiplication) {
-        AutoDiff.Term a, b;
-        if (node.SubtreeCount == 1) {
-          a = AutoDiff.TermBuilder.Constant(1);
-          if (!TryTransformToAutoDiff(node.GetSubtree(0), variables, parameters, variableNames, updateVariableWeights, out b)) {
+        List<AutoDiff.Term> terms = new List<Term>();
+        foreach (var subTree in node.Subtrees) {
+          AutoDiff.Term t;
+          if (!TryTransformToAutoDiff(subTree, variables, parameters, variableNames, updateVariableWeights, out t)) {
             term = null;
             return false;
           }
-        } else {
-          if (!TryTransformToAutoDiff(node.GetSubtree(0), variables, parameters, variableNames, updateVariableWeights, out a) ||
-              !TryTransformToAutoDiff(node.GetSubtree(1), variables, parameters, variableNames, updateVariableWeights, out b)) {
+          terms.Add(t);
+        }
+        if (terms.Count == 1) term = terms[0];
+        else term = terms.Aggregate((a, b) => new AutoDiff.Product(a, b));
+        return true;
+
+      }
+      if (node.Symbol is Division) {
+        List<AutoDiff.Term> terms = new List<Term>();
+        foreach (var subTree in node.Subtrees) {
+          AutoDiff.Term t;
+          if (!TryTransformToAutoDiff(subTree, variables, parameters, variableNames, updateVariableWeights, out t)) {
             term = null;
             return false;
           }
-        }
-        List<AutoDiff.Term> factors = new List<Term>();
-        foreach (var subTree in node.Subtrees.Skip(2)) {
-          AutoDiff.Term f;
-          if (!TryTransformToAutoDiff(subTree, variables, parameters, variableNames, updateVariableWeights, out f)) {
-            term = null;
-            return false;
-          }
-          factors.Add(f);
-        }
-        term = AutoDiff.TermBuilder.Product(a, b, factors.ToArray());
-        return true;
-      }
-      if (node.Symbol is Division) {
-        AutoDiff.Term a, b;
-        if (node.SubtreeCount == 1) {
-          a = AutoDiff.TermBuilder.Constant(1);
-          if (!TryTransformToAutoDiff(node.GetSubtree(0), variables, parameters, variableNames, updateVariableWeights, out b)) {
-            term = null;
-            return false;
-          }
-        } else {
-          if (!TryTransformToAutoDiff(node.GetSubtree(0), variables, parameters, variableNames, updateVariableWeights, out a) ||
-              !TryTransformToAutoDiff(node.GetSubtree(1), variables, parameters, variableNames, updateVariableWeights, out b)) {
-            term = null;
-            return false;
-          }
-        }
-        List<AutoDiff.Term> factors = new List<Term>();
-        foreach (var subTree in node.Subtrees.Skip(2)) {
-          AutoDiff.Term f;
-          if (!TryTransformToAutoDiff(subTree, variables, parameters, variableNames, updateVariableWeights, out f)) {
-            term = null;
-            return false;
-          }
-          factors.Add(1.0 / f);
-        }
-        term = AutoDiff.TermBuilder.Product(a, 1.0 / b, factors.ToArray());
+          terms.Add(t);
+        }
+        if (terms.Count == 1) term = 1.0 / terms[0];
+        else term = terms.Aggregate((a, b) => new AutoDiff.Product(a, 1.0 / b));
         return true;
       }