Skip to content

Commit dea155f

Browse files
committed
Fix interval keyword argument in oedi_9068 gallery example
Fixes #2791. The oedi_9068 gallery example calls get_nsrdb_psm4_conus() with interval=5, but the function's parameter is named time_step (the interval name comes from the NSRDB API query string, not the Python signature). This was missed when PR #2582 migrated the example from get_psm3 (which used interval) to get_nsrdb_psm4_conus (which uses time_step). Changes interval=5 to time_step=5 in the example call. No test changes needed — test_get_nsrdb_psm4_conus_5min already exercises time_step=5 against the function directly.
1 parent 0f7a205 commit dea155f

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

docs/examples/system-models/oedi_9068.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,8 @@
145145
'albedo', 'precipitable_water']
146146
psm3, psm3_metadata = pvlib.iotools.get_nsrdb_psm4_conus(latitude, longitude,
147147
api_key, email,
148-
year=2019, interval=5,
148+
year=2019,
149+
time_step=5,
149150
parameters=keys,
150151
map_variables=True,
151152
leap_day=True)

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@ Enhancements
2323
Documentation
2424
~~~~~~~~~~~~~
2525

26+
* Fixed broken ``interval`` keyword argument in the ``oedi_9068`` gallery
27+
example; the correct parameter name is ``time_step``. (:issue:`2791`)
28+
2629

2730
Testing
2831
~~~~~~~

0 commit comments

Comments
 (0)