pylint

Author: wschoonveld

decoimpact/business/entities/rules/time_aggregation_rule.py

@@ -80,7 +80,6 @@ def filter_years(
         start = str(self._start_year) if self._start_year is not None else None
         end = str(self._end_year) if self._end_year is not None else None
         slice_obj = slice(start, end)
-        # filtered_array = value_array.sel({time_dim_name: slice_obj})
         return value_array.sel({time_dim_name: slice_obj})
 
     def execute(self, value_array: _xr.DataArray, logger: ILogger) -> _xr.DataArray:

decoimpact/data/entities/time_aggregation_rule_data.py

@@ -37,17 +37,17 @@ def __init__(
         self._start_year = start_year
         self._end_year = end_year
 
-    # @property
-    # def input_variable(self) -> str:
-    #     """Name of the input variable"""
-    #     return self._input_variable
-
-    # @property
-    # def start_year(self) -> Optional[int]:
-    #     """Start year for aggregation"""
-    #     return self._start_year
-
-    # @property
-    # def end_year(self) -> Optional[int]:
-    #     """End year for aggregation"""
-    #     return self._end_year
+    @property
+    def input_variable(self) -> str:
+        """Name of the input variable"""
+        return self._input_variable
+
+    @property
+    def start_year(self) -> Optional[int]:
+        """Start year for aggregation"""
+        return self._start_year
+
+    @property
+    def end_year(self) -> Optional[int]:
+        """End year for aggregation"""
+        return self._end_year