Changeset 5361 for branches/HeuristicLab.MetaOptimization/HeuristicLab.Problems.MetaOptimization/3.3/Encoding/ParameterConfigurations/ParameterConfiguration.cs

Timestamp:

01/24/11 10:12:39 (13 years ago)

Author:

cneumuel

Message:

#1215

• minor bugfixes

File:

• branches/HeuristicLab.MetaOptimization/HeuristicLab.Problems.MetaOptimization/3.3/Encoding/ParameterConfigurations/ParameterConfiguration.cs (modified) (3 diffs)

branches/HeuristicLab.MetaOptimization/HeuristicLab.Problems.MetaOptimization/3.3/Encoding/ParameterConfigurations/ParameterConfiguration.cs

@@ -146 +146 @@
       this.ValueConfigurations = new CheckedValueConfigurationList(this.validValues ?? CreateValidValues());
       this.ActualValue = new ConstrainedValue(
-        valueParameter.Value != null ? (IItem)valueParameter.Value.Clone() : null, 
+        valueParameter.Value != null ? valueParameter.Value : null, // don't clone here; otherwise settings of a non-optimized ValueParameter will not be synchronized with the ConstrainedValue
         valueParameter.DataType, 
         this.validValues != null ? new ItemSet<IItem>(this.validValues) : CreateValidValues(), 
@@ -200 +200 @@
       if (this.ActualValue != null) this.ActualValue.ToStringChanged -= new EventHandler(ActualValue_ToStringChanged);
     }
-
+    
     private void PopulateValueConfigurations() {
       foreach (Type t in this.validTypes) {
@@ -370 +370 @@
         if (this.Optimize) {
           var vc = this.ValueConfigurations[actualValueConfigurationIndex];
-          if (IsSubclassOfRawGeneric(typeof(ValueTypeValue<>), vc.GetType())) {
+          if (vc.ActualValue == null || vc.ActualValue.Value == null) {
+            sb.Append(string.Format("{0}: null", parameterName));
+          } else if (IsSubclassOfRawGeneric(typeof(ValueTypeValue<>), vc.ActualValue.Value.GetType())) {
             // for int, double, bool use the value directly
-            sb.Append(string.Format("{0}: {1}", parameterName, this.ActualValue.Value != null ? this.ActualValue.Value.ToString() : "null"));
+            sb.Append(string.Format("{0}: {1}", parameterName, this.ActualValue.Value.ToString()));
           } else {
             // for other types use NumberedName (this also uses the Number-Property for otherwise ambiguous ValueConfigurations)