Skip to content

Commit 69d1bf3

Browse files
mikofskiwholmgren
authored andcommitted
update solar_position docstring (#327)
* update solar_position docstring * change units for pressure to mbars * add note to whatsnew-0.4.5 about mbars in low-level spa.py module ONLY
1 parent 66c70ba commit 69d1bf3

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

docs/sphinx/source/whatsnew/v0.4.5.txt

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,11 @@ Bug fixes
1313
* Added lower accuracy formulas for equation of time, declination, hour angle
1414
and solar zenith.
1515
* Remove all instances of .ix (:issue:`322`)
16+
* Update docstring in `pvlib.spa.solar_position` - change units of pressure to
17+
millibars. *NOTE*: units of pressure in `pvlib.solar_position.spa_python` and
18+
`pvlib.solar_position.spa_c` are still Pascals. This update should have no
19+
effect on most users, since it only applies to the low-level `spa.py` module.
20+
(:issue:`327`)
1621

1722

1823
Enhancements

pvlib/spa.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1103,7 +1103,7 @@ def solar_position(unixtime, lat, lon, elev, pressure, temp, delta_t,
11031103
elev : float
11041104
Elevation of location in meters
11051105
pressure : int or float
1106-
avg. yearly pressure at location in Pascals;
1106+
avg. yearly pressure at location in millibars;
11071107
used for atmospheric correction
11081108
temp : int or float
11091109
avg. yearly temperature at location in

0 commit comments

Comments
 (0)