Changeset 16152 for branches/2925_AutoDiffForDynamicalModels/HeuristicLab.Problems.DynamicalSystemsModelling/3.3/Problem.cs

Timestamp:

09/17/18 18:31:24 (6 years ago)

Author:

gkronber

Message:

#2925: removed obsolete comment and a statement introduced in r16126

File:

• branches/2925_AutoDiffForDynamicalModels/HeuristicLab.Problems.DynamicalSystemsModelling/3.3/Problem.cs (modified) (4 diffs)

branches/2925_AutoDiffForDynamicalModels/HeuristicLab.Problems.DynamicalSystemsModelling/3.3/Problem.cs

@@ -210 +210 @@
       RegisterEventHandlers();
       InitAllParameters();
+
+      // TODO: do not clear selection of target variables when the input variables are changed
+      // TODO: UI hangs when selecting / deselecting input variables because the encoding is updated on each item
     }
 
@@ -424 +427 @@
       results["Prediction (test)"].Value = testList.AsReadOnly();
 
+      #region simplification of models
+      // TODO the dependency of HeuristicLab.Problems.DataAnalysis.Symbolic is not ideal
       var modelList = new ItemList<ISymbolicExpressionTree>();
       foreach (var tree in trees) {
@@ -440 +445 @@
           parentNode.InsertSubtree(replacementIndex, replacementNode);
         }
-        // TODO: simplify trees
 
         modelList.Add(shownTree);
       }
       results["Models"].Value = modelList.AsReadOnly();
+      #endregion
     }
 
@@ -635 +640 @@
     private void OnProblemDataChanged() {
       UpdateTargetVariables();        // implicitly updates other dependent parameters
-      UpdateGrammarAndEncoding();
       var handler = ProblemDataChanged;
       if (handler != null) handler(this, EventArgs.Empty);