question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

More common terms for solar zenith and solar azimuth

See original GitHub issue

In pvlib, two solar zenith quantities are of interest: true (geometric) zenith angle, and apparent (refraction corrected) zenith angle. Currently, zenith, solar_zenith and apparent_zenith all appear in different places, and the meaning of each isn’t consistent throughout. For example, solarposition/spa_c uses zenith (true) and apparent_zenith (refraction-corrected), and Location methods expect these terms. But both apparent_zenith and solar_zenith are used in SingleAxisTracker methods (the latter without being specific in the docstring), and in places where I think refraction-corrected zenith should be expected, e.g., PVSystem.get_aoi.

For solar azimuth, there is not counterpart to true vs. refraction-corrected. But for the solar azimuth, both azimuth and solar_azimuth are being used.

Describe the solution you’d like

  1. Agreement on the pvlib meaning of zenith and solar_zenith. apparent_zenith’s meaning is clear. The other two terms leave room for uncertainty.

  2. Agreement on one term to use for solar azimuth: azimuth or solar_azimuth.

Describe alternatives you’ve considered It makes sense to have a term that is not specific to true or apparent zenith. Perhaps zenith for this use, and solar_zenith specifically for true zenith?

Issue Analytics

  • State:open
  • Created 2 years ago
  • Reactions:2
  • Comments:18 (18 by maintainers)

github_iconTop GitHub Comments

2reactions
cwhansecommented, Mar 2, 2022

I am willing to draft a PR. If we are changing parameter names, what’s the machinery for deprecation? Add the new name and retain the old name, raising a warning if the old name is used?

2reactions
wholmgrencommented, Feb 28, 2022

So are we in agreement with https://github.com/pvlib/pvlib-python/issues/1403#issuecomment-1043363825 except for the geom/analytical name?

I generally agree with @mikofski on the descriptor order but we already have the solar_zenith_analytical function and I think consistency with that is a plus. If we are still looking for a more descriptive name, the function doc string mentions “spherical trigonometry” so maybe solar_zenith_spherical (or spherical_solar_zenith).

Thinking we should finalize this soon so that we can make a 0.9.1 release.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Solar azimuth angle - Wikipedia
The solar azimuth angle is the azimuth (horizontal angle with respect to north) of the Sun's position. This horizontal coordinate defines the Sun's...
Read more >
Solar Azimuth Angle - an overview | ScienceDirect Topics
Solar azimuth angle is defined as the angle between the projection of sun's centre onto the horizontal plane and due south direction. From:...
Read more >
7 Important Solar Angles Nobody Talks About | Solar Labs
A solar energy is one of the most prominent renewable energy source in ... Zenith in astronomy terms means the point in sky...
Read more >
About solar zenith and azimuth - CATALYST.Earth
The solar azimuth and solar zenith express the position of the sun. The solar azimuth is the angle of the direction of the...
Read more >
Tilt & Azimuth Angle: Find the Optimal Angle to Mount Your ...
The “tilt angle” or “elevation angle” describes the vertical angle of your solar panels. “Azimuth angle” is their horizontal facing in relation ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found