Skip to content

Commit c12a477

Browse files
Make dni_extra a required parameter for ghi_from_poa_driesse_2023 (#2331)
* make dni_extra a required parameter * update whatsnew * Update docs/sphinx/source/whatsnew/v0.11.2.rst Co-authored-by: Echedey Luis <[email protected]> --------- Co-authored-by: Echedey Luis <[email protected]>
1 parent 042af42 commit c12a477

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ Bug Fixes
2222
(:issue:`1338`, :pull:`2227`)
2323
* Handle DST transitions that happen at midnight in :py:func:`pvlib.solarposition.hour_angle`
2424
(:issue:`2132` :pull:`2133`)
25+
* Changed ``dni_extra`` to a required parameter in :py:func:`pvlib.irradiance.ghi_from_poa_driesse_2023`
26+
(:issue:`2279` :pull:`2331`)
2527

2628
Bug fixes
2729
~~~~~~~~~
@@ -85,3 +87,4 @@ Contributors
8587
* Echedey Luis (:ghuser:`echedey-ls`)
8688
* Kevin Anderson (:ghuser:`kandersolar`)
8789
* Scott Nelson (:ghuser:`scttnlsn`)
90+
* Ioannis Sifnaios (:ghuser:`IoannisSifnaios`)

pvlib/irradiance.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1528,7 +1528,7 @@ def poa_error(ghi):
15281528
def ghi_from_poa_driesse_2023(surface_tilt, surface_azimuth,
15291529
solar_zenith, solar_azimuth,
15301530
poa_global,
1531-
dni_extra=None, airmass=None, albedo=0.25,
1531+
dni_extra, airmass=None, albedo=0.25,
15321532
xtol=0.01,
15331533
full_output=False):
15341534
'''
@@ -1549,7 +1549,7 @@ def ghi_from_poa_driesse_2023(surface_tilt, surface_azimuth,
15491549
Solar azimuth angle. [degree]
15501550
poa_global : numeric
15511551
Plane-of-array global irradiance, aka global tilted irradiance. [Wm⁻²]
1552-
dni_extra : numeric, optional
1552+
dni_extra : numeric
15531553
Extraterrestrial direct normal irradiance. [Wm⁻²]
15541554
airmass : numeric, optional
15551555
Relative airmass (not adjusted for pressure). [unitless]

0 commit comments

Comments
 (0)