Changeset 17407 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/RegressionProblemDataConverter.cs

Timestamp:

01/27/20 12:18:06 (4 years ago)

Author:

dpiringe

Message:

#3026:

• simplified converter inheritance:
  • BaseConverter now only has Inject and Extract from IJsonItemConverter as abstract methods
  • removed ParameterBaseConverter
  • concrete converters have to initialize their JsonItem now -> enables better handling with different types of JsonItem

File:

• branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/RegressionProblemDataConverter.cs (modified) (2 diffs)

branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/RegressionProblemDataConverter.cs

@@ -14 +14 @@
     public override Type ConvertableType => HEAL.Attic.Mapper.StaticCache.GetType(new Guid("EE612297-B1AF-42D2-BF21-AF9A2D42791C"));
 
-    public override void Populate(IItem value, IJsonItem item, IJsonItemConverter root) {      
+    public override void Inject(IItem item, IJsonItem data, IJsonItemConverter root) {
+      // TODO: inject data
+      throw new NotImplementedException();
+    }
+
+    public override IJsonItem Extract(IItem value, IJsonItemConverter root) {
+      IJsonItem item = new JsonItem() { Name = value.ItemName };
+
       dynamic val = (dynamic)value;
       object dataset = (object)val.Dataset;
@@ -38 +45 @@
         Range = variables.Select(x => x.Value)
       });
-    }
-
-    public override void InjectData(IItem item, IJsonItem data, IJsonItemConverter root) {
-      // TODO: inject data
-      throw new NotImplementedException();
+      return item;
     }
   }