-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
Milestone
Description
The pvlib.tracking.singleaxis
takes input parameters of apparent_zenith
and apparent_azimuth
. Since the solar azimuth angle is not affected by refraction, this parameter name does not make sense. It is the only place in pvlib that apparent_azimuth
is used.
pvlib-python/pvlib/tracking.py
Lines 9 to 11 in a27c686
def singleaxis(apparent_zenith, apparent_azimuth, | |
axis_tilt=0, axis_azimuth=0, max_angle=90, | |
backtrack=True, gcr=2.0/7.0, cross_axis_tilt=0): |
This discussion is related to #1403.
A number of attendees at the 2025 PVSC pvlib tutorial had issues finding the apparent_azimuth
parameter (for good reason), hence I think the inputs should be changed to solar_
, which would also be consistent with most other pvlib functions (even when they require apparent solar angles).
echedey-ls, mikofski, adriesse and kandersolar