Add some initial thoughts and rename value_filler.py to imputer.py

mlpp_lib/value_filler.py → mlpp_lib/imputer.py

@@ -7,10 +7,16 @@
 from typing_extensions import Self
 
 
-class ValueFiller:
+class Imputer:
     """
     Abstract class for filling missing values in a dataset.
     """
+    #TODO: 
+    #    - create subclasses to handle different types of imputing:
+    #        - constant
+    #        - location-based (stations) / elevation
+    #        - based on lead time ? E.g., replace by the mean of the distribution for observed values at such lead time
+    #        - based on valid time -> may be more appropriate than lead time
 
     @abstractmethod
     def fill(self, data: xr.Dataset) -> xr.Dataset:
@@ -29,4 +35,4 @@ class ConstantFiller(ValueFiller):
     fillvalue: dict[str, float] = field(init=True, default=-5)
 
     def fill(self, data: xr.Dataset) -> xr.Dataset:
-        return data.fillna(self.fillvalue)
+        return data.fillna(self.fillvalue)