Appease flake8

pvlib · markcampanelli · May 12, 2024 · May 12, 2024 · May 12, 2024 · May 12, 2024
commit fe7833b567ca15d640dfdbc9e4dc72acd92143f8
diff --git a/pvlib/iam.py b/pvlib/iam.py
@@ -20,7 +20,7 @@
 def get_builtin_models():
    """Return a dictionary of all builtin IAM models."""

    return {
        'ashrae': ashrae,
        'interp': interp,
        'martin_ruiz': martin_ruiz,
@@ -35,7 +35,7 @@
 def get_builtin_direct_models():
    """Return a dictionary of all builtin direct IAM models."""

    return {
        'ashrae': ashrae,
        'interp': interp,
        'martin_ruiz': martin_ruiz,
@@ -668,7 +668,7 @@
        for Diffuse radiation on Inclined photovoltaic Surfaces (FEDIS)",
        Renewable and Sustainable Energy Reviews, vol. 161, 112362. June 2022.
        :doi:`10.1016/j.rser.2022.112362`
-    
+
     See Also
     --------
     pvlib.iam.ashrae
@@ -1036,8 +1036,8 @@
 
     if passed_params != exp_params:
         raise ValueError(
-            f"The {builtin_model_name} model was expecting to be passed {exp_params},"
-            f"but was handed {passed_params}"
+            f"The {builtin_model_name} model was expecting to be passed"
+            f"{exp_params}, but was handed {passed_params}"
         )
 
 

diff --git a/pvlib/modelchain.py b/pvlib/modelchain.py
@@ -269,7 +269,7 @@ def _head(obj):
                  '\n')
         lines = []
         for attr in mc_attrs:
-            if not (attr.startswith('_') or attr=='times'):
+            if not (attr.startswith('_') or attr == 'times'):
                 lines.append(f' {attr}: ' + _mcr_repr(getattr(self, attr)))
         desc4 = '\n'.join(lines)
         return (desc1 + desc2 + desc3 + desc4)
@@ -386,7 +386,6 @@ def __init__(self, system, location,
 
         self.results = ModelChainResult()
 
-
     @classmethod
     def with_pvwatts(cls, system, location,
                      clearsky_model='ineichen',
@@ -785,8 +784,9 @@ def aoi_model(self, model):
         else:
             self._aoi_model = partial(model, self)
 
+    # FIXME What about diffuse versions of models? What about schlick,
+    # schlick_diffuse, and custom models?
 
-    # FIXME What about diffuse versions of models? What about schlick and custom models?
     def infer_aoi_model(self):
         module_parameters = tuple(
             array.module_parameters for array in self.system.arrays)
@@ -805,13 +805,14 @@ def infer_aoi_model(self):
         elif iam_model_params['interp'] <= params:
             return self.interp_aoi_loss
 
-        raise ValueError('could not infer AOI model from '
-                            'system.arrays[i].module_parameters. Check that '
-                            'the module_parameters for all Arrays in '
-                            'system.arrays contain parameters for the '
-                            'physical, aoi, ashrae, martin_ruiz or interp '
-                            'model; explicitly set the model with the '
-                            'aoi_model kwarg; or set aoi_model="no_loss".')
+        raise ValueError(
+            'could not infer AOI model from '
+            'system.arrays[i].module_parameters. Check that the '
+            'module_parameters for all Arrays in system.arrays contain '
+            'parameters for the physical, aoi, ashrae, martin_ruiz or interp '
+            'model; explicitly set the model with the aoi_model kwarg; or '
+            'set aoi_model="no_loss".'
+        )
 
     def ashrae_aoi_loss(self):
         self.results.aoi_modifier = self.system.get_iam(