-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add reference for Gueymard 93 relative airmass model #2424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 5 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
.. _whatsnew_01201: | ||
|
||
|
||
v0.12.1 (XXXX, 2025) | ||
------------------------ | ||
|
||
Breaking Changes | ||
~~~~~~~~~~~~~~~~ | ||
|
||
|
||
Bug fixes | ||
~~~~~~~~~ | ||
|
||
|
||
Enhancements | ||
~~~~~~~~~~~~ | ||
|
||
|
||
Documentation | ||
~~~~~~~~~~~~~ | ||
* Add a supporting reference to :py:func:`pvlib.atmosphere.get_relative_airmass` (:issue:`2390`, :pull:`2424`) | ||
|
||
Testing | ||
~~~~~~~ | ||
|
||
|
||
Maintenance | ||
~~~~~~~~~~~ | ||
|
||
|
||
Contributors | ||
~~~~~~~~~~~~ | ||
* Cliff Hansen (:ghuser:`cwhanse`) |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -147,20 +147,13 @@ def get_relative_airmass(zenith, model='kastenyoung1989'): | |||||
|
||||||
* 'simple' - secant(apparent zenith angle) - | ||||||
Note that this gives -Inf at zenith=90 | ||||||
* 'kasten1966' - See reference [1] - | ||||||
requires apparent sun zenith | ||||||
* 'youngirvine1967' - See reference [2] - | ||||||
requires true sun zenith | ||||||
* 'kastenyoung1989' (default) - See reference [3] - | ||||||
requires apparent sun zenith | ||||||
* 'gueymard1993' - See reference [4] - | ||||||
requires apparent sun zenith | ||||||
* 'young1994' - See reference [5] - | ||||||
requires true sun zenith | ||||||
* 'pickering2002' - See reference [6] - | ||||||
requires apparent sun zenith | ||||||
* 'gueymard2003' - See references [7] and [8] - | ||||||
requires apparent sun zenith | ||||||
* 'kasten1966' - See [1]_ - requires apparent sun zenith | ||||||
* 'youngirvine1967' - See [2]_ - requires true sun zenith | ||||||
* 'kastenyoung1989' (default) - See [3]_ - requires apparent sun zenith | ||||||
* 'gueymard1993' - See [4]_, [5]_ - requires apparent sun zenith | ||||||
* 'young1994' - See [6]_ - requires true sun zenith | ||||||
* 'pickering2002' - See [7]_ - requires apparent sun zenith | ||||||
* 'gueymard2003' - See [8]_, [9]_ - requires apparent sun zenith | ||||||
|
||||||
Returns | ||||||
------- | ||||||
|
@@ -174,41 +167,56 @@ def get_relative_airmass(zenith, model='kastenyoung1989'): | |||||
other models use true (not refraction-adjusted) zenith angle. Apparent | ||||||
zenith angles should be calculated at sea level. | ||||||
|
||||||
Comparison among several models is reported in [10]_. | ||||||
|
||||||
References | ||||||
---------- | ||||||
.. [1] Fritz Kasten. "A New Table and Approximation Formula for the | ||||||
Relative Optical Air Mass". Technical Report 136, Hanover, N.H.: | ||||||
U.S. Army Material Command, CRREL. | ||||||
.. [1] Fritz Kasten, "A New Table and Approximation Formula for the | ||||||
Relative Optical Air Mass," CRREL (U.S. Army), Hanover, NH, USA, | ||||||
Technical Report 136, 1965. | ||||||
:doi:`11681/5671` | ||||||
|
||||||
.. [2] A. T. Young and W. M. Irvine, "Multicolor Photoelectric | ||||||
Photometry of the Brighter Planets," The Astronomical Journal, vol. | ||||||
72, pp. 945-950, 1967. | ||||||
:doi:`10.2172/110366` | ||||||
|
:doi:`10.2172/110366` | |
:doi:`10.1086/110366` |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fo needs to be of (for some reason it won't let me suggest it)
Could consider adding the url: http://dioi.org/jc01.pdf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clarification for users out of context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought just "reference" was okay without "bibliographic". Adding the
model ``gueymard1993`` in
seems reasonable thoughThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I sometimes think of code references, but given this piece of description is in the Docs section, you are completely right.