Skip to content

Commit c34058d

Browse files
eeshsaxenakandersolarAdamRJensen
authored
docs: fix parameter name mismatches in docstrings (#2804)
* docs: fix parameter name mismatches in docstrings - pvsystem.py: `aoi_model` -> `iam_model` in PVSystem.get_iam and Array.get_iam (signature parameter is `iam_model`) - solarposition.py: `time` -> `times` in sun_rise_set_transit_ephem - irradiance.py: _liujordan documented a nonexistent `pressure` parameter; replace with the actual `airmass` parameter - bifacial/infinite_sheds.py: remove duplicated stale Parameters block in _poa_sky_diffuse_pv (listed `f_x` and `npoints` which are not parameters of this function) * Add contributor entry to v0.15.3 whatsnew * Update pvlib/irradiance.py Co-authored-by: Adam R. Jensen <39184289+AdamRJensen@users.noreply.github.com> --------- Co-authored-by: Kevin Anderson <kevin.anderso@gmail.com> Co-authored-by: Adam R. Jensen <39184289+AdamRJensen@users.noreply.github.com>
1 parent c4b9ad8 commit c34058d

5 files changed

Lines changed: 6 additions & 21 deletions

File tree

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,5 +45,6 @@ Maintenance
4545

4646
Contributors
4747
~~~~~~~~~~~~
48+
* Eesh Saxena (:ghuser:`eeshsaxena`)
4849
* Karl Hill (:ghuser:`karlhillx`)
4950
* Yonry Zhu (:ghuser:`yonryzhu`)

pvlib/bifacial/infinite_sheds.py

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -45,19 +45,6 @@ def _poa_sky_diffuse_pv(dhi, gcr, surface_tilt):
4545
Integrated view factors from the shaded and unshaded parts of
4646
the row slant height to the sky.
4747
48-
Parameters
49-
----------
50-
f_x : numeric
51-
Fraction of row slant height from the bottom that is shaded from
52-
direct irradiance. [unitless]
53-
surface_tilt : numeric
54-
Surface tilt angle in degrees from horizontal, e.g., surface facing up
55-
= 0, surface facing horizon = 90. [degree]
56-
gcr : float
57-
Ratio of row slant length to row spacing (pitch). [unitless]
58-
npoints : int, default 100
59-
Number of points for integration. [unitless]
60-
6148
A detailed calculation would be
6249
6350
dhi * (f_x * vf_shade_sky_integ + (1 - f_x) * vf_noshade_sky_integ)
@@ -73,9 +60,6 @@ def _poa_sky_diffuse_pv(dhi, gcr, surface_tilt):
7360
7461
Parameters
7562
----------
76-
f_x : numeric
77-
Fraction of row slant height from the bottom that is shaded from
78-
direct irradiance. [unitless]
7963
dhi : numeric
8064
Diffuse horizontal irradiance (DHI). [W/m^2]
8165
gcr : float

pvlib/irradiance.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3122,8 +3122,8 @@ def _liujordan(zenith, transmittance, airmass, dni_extra=1367.0):
31223122
transmittance: float
31233123
Atmospheric transmittance between 0 and 1.
31243124
3125-
pressure: float, default 101325.0
3126-
Air pressure
3125+
airmass: numeric
3126+
Optical air mass. [unitless]
31273127
31283128
dni_extra: float, default 1367.0
31293129
Direct irradiance incident at the top of the atmosphere.

pvlib/pvsystem.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ def get_iam(self, aoi, iam_model='physical'):
395395
aoi : numeric or tuple of numeric
396396
The angle of incidence in degrees.
397397
398-
aoi_model : string, default 'physical'
398+
iam_model : string, default 'physical'
399399
The IAM model to be used. Valid strings are 'physical', 'ashrae',
400400
'martin_ruiz', 'sapm' and 'interp'.
401401
Returns
@@ -1188,7 +1188,7 @@ def get_iam(self, aoi, iam_model='physical'):
11881188
aoi : numeric
11891189
The angle of incidence in degrees.
11901190
1191-
aoi_model : string, default 'physical'
1191+
iam_model : string, default 'physical'
11921192
The IAM model to be used. Valid strings are 'physical', 'ashrae',
11931193
'martin_ruiz', 'sapm' and 'interp'.
11941194

pvlib/solarposition.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -516,7 +516,7 @@ def sun_rise_set_transit_ephem(times, latitude, longitude,
516516
517517
Parameters
518518
----------
519-
time : pandas.DatetimeIndex
519+
times : pandas.DatetimeIndex
520520
Must be localized
521521
latitude : float
522522
Latitude in degrees, positive north of equator, negative to south

0 commit comments

Comments
 (0)