Skip to content

Commit 10edc65

Browse files
RDaxinikandersolar
andauthored
Fix typos in ModelChain error messages (#2492)
* aoi->sapm * other misc typos * grammar, unit * Update pvlib/modelchain.py Co-authored-by: Kevin Anderson <[email protected]> --------- Co-authored-by: Kevin Anderson <[email protected]>
1 parent dfafe6d commit 10edc65

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

pvlib/modelchain.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
# 'cell_temperature' overrides ModelChain.temperature_model and sets
3232
# ModelChain.cell_temperature to the data. If 'module_temperature' is provided,
3333
# overrides ModelChain.temperature_model with
34-
# pvlib.temperature.sapm_celL_from_module
34+
# pvlib.temperature.sapm_cell_from_module
3535
TEMPERATURE_KEYS = ('module_temperature', 'cell_temperature')
3636

3737
DATA_KEYS = WEATHER_KEYS + POA_KEYS + TEMPERATURE_KEYS
@@ -46,7 +46,7 @@
4646
# for Flat-Plate Photovoltaic Arrays. SAND85-0330. Albuquerque, NM:
4747
# Sandia National Laboratories. Accessed September 3, 2013:
4848
# http://prod.sandia.gov/techlib/access-control.cgi/1985/850330.pdf
49-
# pvlib python does not implement that model, so use the SAPM instead.
49+
# pvlib-python does not implement that model, so it uses the SAPM instead.
5050
PVWATTS_CONFIG = dict(
5151
dc_model='pvwatts', ac_model='pvwatts', losses_model='pvwatts',
5252
transposition_model='perez', aoi_model='physical',
@@ -84,7 +84,7 @@ def get_orientation(strategy, **kwargs):
8484
surface_tilt = 0
8585
else:
8686
raise ValueError('invalid orientation strategy. strategy must '
87-
'be one of south_at_latitude, flat,')
87+
'be one of south_at_latitude_tilt, flat,')
8888

8989
return surface_tilt, surface_azimuth
9090

@@ -162,8 +162,8 @@ class ModelChainResult:
162162
# per DC array information
163163
total_irrad: Optional[PerArray[pd.DataFrame]] = field(default=None)
164164
""" DataFrame (or tuple of DataFrame, one for each array) containing
165-
columns ``'poa_global'``, ``'poa_direct'`` ``'poa_diffuse'``,
166-
``poa_sky_diffuse'``, ``'poa_ground_diffuse'`` (W/m2); see
165+
columns ``'poa_global'``, ``'poa_direct'``, ``'poa_diffuse'``,
166+
``poa_sky_diffuse'``, and ``'poa_ground_diffuse'`` (Wm⁻²); see
167167
:py:func:`~pvlib.irradiance.get_total_irradiance` for details.
168168
"""
169169

@@ -190,12 +190,12 @@ class ModelChainResult:
190190

191191
cell_temperature: Optional[PerArray[pd.Series]] = field(default=None)
192192
"""Series (or tuple of Series, one for each array) containing cell
193-
temperature (C).
193+
temperature (°C).
194194
"""
195195

196196
effective_irradiance: Optional[PerArray[pd.Series]] = field(default=None)
197197
"""Series (or tuple of Series, one for each array) containing effective
198-
irradiance (W/m2) which is total plane-of-array irradiance adjusted for
198+
irradiance (Wm⁻²) which is total plane-of-array irradiance adjusted for
199199
reflections and spectral content.
200200
"""
201201

@@ -215,12 +215,12 @@ class ModelChainResult:
215215

216216
dc_ohmic_losses: Optional[PerArray[pd.Series]] = field(default=None)
217217
"""Series (or tuple of Series, one for each array) containing DC ohmic
218-
loss (W) calculated by ``ModelChain.dc_ohmic_model``.
218+
losses (W) calculated by ``ModelChain.dc_ohmic_model``.
219219
"""
220220

221221
# copies of input data, for user convenience
222222
weather: Optional[PerArray[pd.DataFrame]] = None
223-
"""DataFrame (or tuple of DataFrame, one for each array) contains a
223+
"""DataFrame (or tuple of DataFrame, one for each array) containing a
224224
copy of the input weather data.
225225
"""
226226

@@ -806,7 +806,7 @@ def infer_aoi_model(self):
806806
'system.arrays[i].module_parameters. Check that '
807807
'the module_parameters for all Arrays in '
808808
'system.arrays contain parameters for the '
809-
'physical, aoi, ashrae, martin_ruiz or interp '
809+
'physical, sapm, ashrae, martin_ruiz or interp '
810810
'model; explicitly set the model with the '
811811
'aoi_model kwarg; or set aoi_model="no_loss".')
812812

0 commit comments

Comments
 (0)