Changeset 1208

Timestamp:

02/05/09 12:54:52 (16 years ago)

Author:

gkronber

Message:

fixed #487 (GPOperatorLibrary/GPOperatorGroup doesn't work in ThreadParallelEngines) in CEDMA refactoring branch

File:

• branches/CEDMA-Refactoring-Ticket419/HeuristicLab.GP.StructureIdentification/FunctionLibraryInjector.cs (modified) (5 diffs)

branches/CEDMA-Refactoring-Ticket419/HeuristicLab.GP.StructureIdentification/FunctionLibraryInjector.cs

@@ -36 +36 @@
     private const string FUNCTIONLIBRARY = "FunctionLibrary";
 
-    private StructId.Variable variable;
-    private GPOperatorLibrary operatorLibrary;
-
     public override string Description {
       get { return @"Injects a default function library for regression and classification problems."; }
@@ -51 +48 @@
 
     public override IOperation Apply(IScope scope) {
+      StructId.Variable variable;
+      GPOperatorLibrary operatorLibrary;
+
       ItemList<IntData> allowedFeatures = GetVariableValue<ItemList<IntData>>(ALLOWEDFEATURES, scope, true);
       int targetVariable = GetVariableValue<IntData>(TARGETVARIABLE, scope, true).Data;
@@ -56 +56 @@
       // remove the target-variable in case it occures in allowed features
       List<IntData> ts = allowedFeatures.FindAll(d => d.Data == targetVariable);
-      foreach(IntData t in ts) allowedFeatures.Remove(t);
-
-      InitDefaultOperatorLibrary();
-
-      int[] allowedIndexes = new int[allowedFeatures.Count];
-      for(int i = 0; i < allowedIndexes.Length; i++) {
-        allowedIndexes[i] = allowedFeatures[i].Data;
-      }
-
-      variable.SetConstraints(allowedIndexes, 0, 0);
-
-      scope.AddVariable(new HeuristicLab.Core.Variable(scope.TranslateName(FUNCTIONLIBRARY), operatorLibrary));
-      return null;
-    }
-
-    private void InitDefaultOperatorLibrary() {
+      foreach (IntData t in ts) allowedFeatures.Remove(t);
+
       variable = new StructId.Variable();
       StructId.Constant constant = new StructId.Constant();
@@ -173 +159 @@
       operatorLibrary.GPOperatorGroup.AddOperator(tangens);
       operatorLibrary.GPOperatorGroup.AddOperator(xor);
+
+      int[] allowedIndexes = new int[allowedFeatures.Count];
+      for (int i = 0; i < allowedIndexes.Length; i++) {
+        allowedIndexes[i] = allowedFeatures[i].Data;
+      }
+
+      variable.SetConstraints(allowedIndexes, 0, 0);
+
+      scope.AddVariable(new HeuristicLab.Core.Variable(scope.TranslateName(FUNCTIONLIBRARY), operatorLibrary));
+      return null;
     }
 
     private void SetAllowedSubOperators(IFunction f, IFunction[] gs) {
-      foreach(IConstraint c in f.Constraints) {
-        if(c is SubOperatorTypeConstraint) {
+      foreach (IConstraint c in f.Constraints) {
+        if (c is SubOperatorTypeConstraint) {
           SubOperatorTypeConstraint typeConstraint = c as SubOperatorTypeConstraint;
           typeConstraint.Clear();
-          foreach(IFunction g in gs) {
+          foreach (IFunction g in gs) {
             typeConstraint.AddOperator(g);
           }
-        } else if(c is AllSubOperatorsTypeConstraint) {
+        } else if (c is AllSubOperatorsTypeConstraint) {
           AllSubOperatorsTypeConstraint typeConstraint = c as AllSubOperatorsTypeConstraint;
           typeConstraint.Clear();
-          foreach(IFunction g in gs) {
+          foreach (IFunction g in gs) {
             typeConstraint.AddOperator(g);
           }
@@ -194 +190 @@
 
     private void SetAllowedSubOperators(IFunction f, int p, IFunction[] gs) {
-      foreach(IConstraint c in f.Constraints) {
-        if(c is SubOperatorTypeConstraint) {
+      foreach (IConstraint c in f.Constraints) {
+        if (c is SubOperatorTypeConstraint) {
           SubOperatorTypeConstraint typeConstraint = c as SubOperatorTypeConstraint;
-          if(typeConstraint.SubOperatorIndex.Data == p) {
+          if (typeConstraint.SubOperatorIndex.Data == p) {
             typeConstraint.Clear();
-            foreach(IFunction g in gs) {
+            foreach (IFunction g in gs) {
               typeConstraint.AddOperator(g);
             }