review

Author: cwhanse

docs/sphinx/source/whatsnew/v0.11.3.rst

@@ -11,7 +11,7 @@ Breaking Changes
   pvlib.location.Location.tz attribute. (:issue:`2340`, :pull:`2341`)
 * Users must now provide ModelChain.spectral_model, or the 'no_loss' spectral
   model is assumed. pvlib.modelchain.ModelChain no longer attempts to infer
-  the spectral from PVSystem attributes. (:issue:`2017`, :pull:`2253`)
+  the spectral model from PVSystem attributes. (:issue:`2017`, :pull:`2253`)
 
 Bug fixes
 ~~~~~~~~~

pvlib/modelchain.py

@@ -330,15 +330,15 @@ class ModelChain:
         'interp' and 'no_loss'. The ModelChain instance will be passed as the
         first argument to a user-defined function.
 
-    spectral_model : str or function, default ``'no_loss'``
+    spectral_model : str or function, optional
         Valid strings are:
 
         - ``'sapm'``
         - ``'first_solar'``
         - ``'no_loss'``
 
         The ModelChain instance will be passed as the first argument to
-        a user-defined function.
+        a user-defined function. If not specified, ``'no_loss'`` is assumed.
 
     temperature_model : str or function, optional
         Valid strings are: 'sapm', 'pvsyst', 'faiman', 'fuentes', 'noct_sam'.
@@ -364,7 +364,7 @@ def __init__(self, system, location,
                  solar_position_method='nrel_numpy',
                  airmass_model='kastenyoung1989',
                  dc_model=None, ac_model=None, aoi_model=None,
-                 spectral_model='no_loss', temperature_model=None,
+                 spectral_model=None, temperature_model=None,
                  dc_ohmic_model='no_loss',
                  losses_model='no_loss', name=None):