Skip to content

Commit 09c6fb2

Browse files
authored
Revert solar_zenith to apparent_zenith
1 parent 2a0fb08 commit 09c6fb2

6 files changed

Lines changed: 14 additions & 19 deletions

File tree

docs/examples/agrivoltaics/plot_agrivoltaics_ground_irradiance.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
# of the trackers. For a single-axis tracker, this can be calculated as:
5959

6060
tracking_orientations = pvlib.tracking.singleaxis(
61-
solar_zenith=solpos['apparent_zenith'],
61+
apparent_zenith=solpos['apparent_zenith'],
6262
solar_azimuth=solpos['azimuth'],
6363
axis_azimuth=axis_azimuth,
6464
max_angle=max_angle,

docs/examples/shading/plot_martinez_shade_loss.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@
8989
) # unpack for better readability
9090

9191
tracking_result = pvlib.tracking.singleaxis(
92-
solar_zenith=solar_apparent_zenith,
92+
apparent_zenith=solar_apparent_zenith,
9393
solar_azimuth=solar_azimuth,
9494
axis_tilt=axis_tilt,
9595
axis_azimuth=axis_azimuth,

docs/examples/solar-tracking/plot_single_axis_tracking.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
solpos = solarposition.get_solarposition(times, lat, lon)
3333

3434
truetracking_angles = tracking.singleaxis(
35-
solar_zenith=solpos['apparent_zenith'],
35+
apparent_zenith=solpos['apparent_zenith'],
3636
solar_azimuth=solpos['azimuth'],
3737
axis_tilt=0,
3838
axis_azimuth=180,
@@ -60,7 +60,7 @@
6060

6161
for gcr in [0.2, 0.4, 0.6]:
6262
backtracking_angles = tracking.singleaxis(
63-
solar_zenith=solpos['apparent_zenith'],
63+
apparent=solpos['apparent_zenith'],
6464
solar_azimuth=solpos['azimuth'],
6565
axis_tilt=0,
6666
axis_azimuth=180,

docs/examples/solar-tracking/plot_single_axis_tracking_on_sloped_terrain.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@
9999
fig, ax = plt.subplots()
100100
for cross_axis_tilt in [0, 5, 10]:
101101
tracker_data = tracking.singleaxis(
102-
solar_zenith=solpos['apparent_zenith'],
102+
apparent=solpos['apparent_zenith'],
103103
solar_azimuth=solpos['azimuth'],
104104
axis_tilt=0, # flat because the axis is perpendicular to the slope
105105
axis_azimuth=180, # N-S axis, azimuth facing south
@@ -155,7 +155,7 @@
155155
# before:
156156

157157
tracker_data = tracking.singleaxis(
158-
solar_zenith=solpos['apparent_zenith'],
158+
apparent=solpos['apparent_zenith'],
159159
solar_azimuth=solpos['azimuth'],
160160
axis_tilt=axis_tilt, # no longer flat because the terrain imparts a tilt
161161
axis_azimuth=axis_azimuth,

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@ Breaking Changes
1010

1111
Deprecations
1212
~~~~~~~~~~~~
13-
* Rename parameter name ``aparent_zenith`` to ``solar_zenith`` and ``aparent_azimuth``
14-
to ``solar_azimuth`` in :py:func:`~pvlib.tracking.singleaxis`.
13+
* Rename parameter name ``aparent_azimuth`` to ``solar_azimuth`` in :py:func:`~pvlib.tracking.singleaxis`.
1514
(:issue:`2479`, :pull:`2480`)
1615

1716
Bug fixes

pvlib/tracking.py

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,6 @@
77
from pvlib._deprecation import renamed_kwarg_warning
88

99

10-
@renamed_kwarg_warning(
11-
since='0.13.1',
12-
old_param_name='apparent_zenith',
13-
new_param_name='solar_zenith')
1410
@renamed_kwarg_warning(
1511
since='0.13.1',
1612
old_param_name='apparent_azimuth',
@@ -39,7 +35,7 @@ def singleaxis(solar_zenith, solar_azimuth,
3935
4036
Parameters
4137
----------
42-
solar_zenith : float, 1d array, or Series
38+
apparent_zenith : float, 1d array, or Series
4339
Solar apparent zenith angles in decimal degrees.
4440
4541
solar_azimuth : float, 1d array, or Series
@@ -126,14 +122,14 @@ def singleaxis(solar_zenith, solar_azimuth,
126122
# MATLAB to Python conversion by
127123
# Will Holmgren (@wholmgren), U. Arizona. March, 2015.
128124

129-
if isinstance(solar_zenith, pd.Series):
130-
index = solar_zenith.index
125+
if isinstance(apparent_zenith, pd.Series):
126+
index = apparent_zenith.index
131127
else:
132128
index = None
133129

134130
# convert scalars to arrays
135131
solar_azimuth = np.atleast_1d(solar_azimuth)
136-
solar_zenith = np.atleast_1d(solar_zenith)
132+
apparent_zenith = np.atleast_1d(apparent_zenith)
137133

138134
if solar_azimuth.ndim > 1 or solar_zenith.ndim > 1:
139135
raise ValueError('Input dimensions must not exceed 1')
@@ -150,12 +146,12 @@ def singleaxis(solar_zenith, solar_azimuth,
150146
omega_ideal = shading.projected_solar_zenith_angle(
151147
axis_tilt=axis_tilt,
152148
axis_azimuth=axis_azimuth,
153-
solar_zenith=solar_zenith,
149+
solar_zenith=apparent_zenith,
154150
solar_azimuth=solar_azimuth,
155151
)
156152

157153
# filter for sun above panel horizon
158-
zen_gt_90 = solar_zenith > 90
154+
zen_gt_90 = apparent_zenith > 90
159155
omega_ideal[zen_gt_90] = np.nan
160156

161157
# Account for backtracking
@@ -200,7 +196,7 @@ def singleaxis(solar_zenith, solar_azimuth,
200196
surface_tilt = surface['surface_tilt']
201197
surface_azimuth = surface['surface_azimuth']
202198
aoi = irradiance.aoi(surface_tilt, surface_azimuth,
203-
solar_zenith, solar_azimuth)
199+
apparent_zenith, solar_azimuth)
204200

205201
# Bundle DataFrame for return values and filter for sun below horizon.
206202
out = {'tracker_theta': tracker_theta, 'aoi': aoi,

0 commit comments

Comments
 (0)