Manage todos

Author: SabineHaas

windpowerlib/power_curves.py

@@ -16,7 +16,6 @@ def smooth_power_curve(power_curve_wind_speeds, power_curve_values,
                        block_width=0.5,
                        standard_deviation_method='turbulence_intensity',
                        mean_gauss=0, **kwargs):
-    # TODO: All functions in this module have to work without pandas
     r"""
     Smoothes the input power curve values by using a gaussian distribution.
 
@@ -259,6 +258,8 @@ def density_correct_power_curve(density, power_curve_wind_speeds,
             at Reiner Lemoine Institute, 2014, p. 13
 
     """
+    # TODO: is this error raising needed? Density does not have default value
+    # e.g. there would be an error message also without the next lines.
     if density is None:
         raise TypeError("`density` is None. For the calculation with a " +
                         "density corrected power curve mean density at hub " +

windpowerlib/power_output.py

@@ -226,6 +226,8 @@ def power_curve_density_correction(wind_speed, power_curve_wind_speeds,
             at Reiner Lemoine Institute, 2014, p. 13
 
     """
+    # TODO: is this error raising needed? Density does not have default value
+    # e.g. there would be an error message also without the next lines.
     if density is None:
         raise TypeError("`density` is None. For the calculation with a " +
                         "density corrected power curve density at hub " +

windpowerlib/turbine_cluster_modelchain.py

@@ -164,7 +164,7 @@ def wind_farm_power_curve(self, wind_farm, **kwargs):
             if self. density_correction:
                 pass # TODO: restrictions (density needed)
             if self.wake_losses_method is not None:
-                if self.wind_object.efficiency is None: # TODO if not...
+                if wind_farm.efficiency is None:
                     raise KeyError(
                         "wind_farm_efficiency is needed if " +
                         "`wake_losses_method´ is '{0}', but ".format(