Bug in get_pvgis_hourly related to optimal tilt (#1356)

* Bug in get_pvgis_hourly related to optimal tilt

Currently, the `optimalangles` boolean input argument was passed correctly to the `optimalangles` parameter but also by mistake to `optimalinclination`. Instead the boolean argument `optimal_surface_tilt` should be passed to the `optimalinclination` parameter.

* Fix test url

* Update whatsnew

* Update whatsnew

Co-authored-by: Kevin Anderson <[email protected]>

Co-authored-by: Kevin Anderson <[email protected]>

Author: AdamRJensen

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

@@ -18,6 +18,8 @@ Bug fixes
   (:issue:`1311`, :pull:`1315`)
 * Fixed a bug in :py:func:`pvlib.spectrum.spectrl2` where negative spectral irradiance
   values were returned when the sun is behind the plane of array (:issue:`1348`, :pull:`1349`)
+* Fixed bug in :py:func:`pvlib.iotools.get_pvgis_hourly` where the ``optimal_surface_tilt``
+  argument was not being passed to the ``optimalinclination`` request parameter (:pull:`1356`)
 
 
 Testing
@@ -36,3 +38,4 @@ Contributors
 * :ghuser:`Carlosbogo`
 * Christian Weickhmann (:ghuser:`cweickhmann`)
 * Kevin Anderson (:ghuser:`kanderso-nrel`)
+* Adam R. Jensen (:ghuser:`AdamRJensen`)

pvlib/iotools/pvgis.py

@@ -190,7 +190,7 @@ def get_pvgis_hourly(latitude, longitude, start=None, end=None,
               'trackingtype': trackingtype, 'components': int(components),
               'usehorizon': int(usehorizon),
               'optimalangles': int(optimalangles),
-              'optimalinclination': int(optimalangles), 'loss': loss}
+              'optimalinclination': int(optimal_surface_tilt), 'loss': loss}
     # pvgis only takes 0 for False, and 1 for True, not strings
     if userhorizon is not None:
         params['userhorizon'] = ','.join(str(x) for x in userhorizon)

pvlib/tests/iotools/test_pvgis.py

@@ -273,7 +273,7 @@ def test_get_pvgis_hourly_bad_outputformat(requests_mock):
         get_pvgis_hourly(latitude=45, longitude=8, outputformat='basic')
 
 
-url_additional_inputs = 'https://re.jrc.ec.europa.eu/api/seriescalc?lat=55.6814&lon=12.5758&outputformat=csv&angle=0&aspect=0&pvcalculation=1&pvtechchoice=crystSi&mountingplace=free&trackingtype=0&components=1&usehorizon=1&optimalangles=1&optimalinclination=1&loss=2&userhorizon=10%2C15%2C20%2C10&peakpower=5'  # noqa: E501
+url_additional_inputs = 'https://re.jrc.ec.europa.eu/api/seriescalc?lat=55.6814&lon=12.5758&outputformat=csv&angle=0&aspect=0&pvcalculation=1&pvtechchoice=crystSi&mountingplace=free&trackingtype=0&components=1&usehorizon=1&optimalangles=1&optimalinclination=0&loss=2&userhorizon=10%2C15%2C20%2C10&peakpower=5'  # noqa: E501
 
 
 def test_get_pvgis_hourly_additional_inputs(requests_mock):