Skip to content

Commit 9b8d32d

Browse files
Apply suggestions from code review
Co-authored-by: Kevin Anderson <kevin.anderso@gmail.com>
1 parent 6f18103 commit 9b8d32d

1 file changed

Lines changed: 11 additions & 3 deletions

File tree

pvlib/iotools/meteonorm.py

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -402,8 +402,8 @@ def get_meteonorm_tmy(
402402
surface_tilt=0, surface_azimuth=180,
403403
time_step="1h", horizon="auto", terrain_situation="open",
404404
albedo=None, turbidity="auto", random_seed=None,
405-
clear_sky_radiation_model="esra", future_scenario=None,
406-
future_year=None, interval_index=False,
405+
clear_sky_radiation_model="esra", data_version=None,
406+
future_scenario=None, future_year=None, interval_index=False,
407407
map_variables=True, url=URL):
408408
"""
409409
Retrieve TMY irradiance and weather data from Meteonorm.
@@ -448,6 +448,8 @@ def get_meteonorm_tmy(
448448
with the same random seed will yield identical results.
449449
clear_sky_radiation_model : str, default : 'esra'
450450
Which clearsky model to use. Must be either `'esra'` or `'solis'`.
451+
data_version : str, optional
452+
Deprecated parameter. Has no effect.
451453
future_scenario : str, optional
452454
Future climate scenario.
453455
future_year : int, optional
@@ -491,7 +493,13 @@ def get_meteonorm_tmy(
491493
<https://docs.meteonorm.com/docs/getting-started>`_
492494
.. [3] `Meteonorm API reference
493495
<https://docs.meteonorm.com/api>`_
494-
"""
496+
if data_version is not None:
497+
msg = (
498+
"This parameter was removed from the Meteonorm API "
499+
"and now has no effect."
500+
)
501+
warn_deprecated(since="0.15.2", removal="0.16.0", name="data_version",
502+
addendum=msg)
495503
additional_params = {
496504
"situation": terrain_situation,
497505
"turbidity": turbidity,

0 commit comments

Comments
 (0)