8000 Improve docstring for get_ground_diffuse by cwhanse · Pull Request #1953 · pvlib/pvlib-python · GitHub
[go: up one dir, main page]
Skip to content

Improve docstring for get_ground_diffuse #1953

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.

Sign up for GitHub

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

Merged
merged 11 commits into from
Feb 27, 2024
Merged

Improve docstring for get_ground_diffuse #1953

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
d0173e1
improve docstring for get_ground_diffuse
cwhanse Jan 26, 2024
5f23a6e
fix formatting. Reduce description text. Whatsnew
cwhanse Jan 26, 2024
1bd9fb8
try to get the sphinx reference
cwhanse Jan 26, 2024
ea83426
sphinx reference
cwhanse Jan 26, 2024
43c346f
use doi
cwhanse Jan 26, 2024
2114c42
use math
cwhanse Jan 29, 2024
5bcb727
use math
cwhanse Jan 29, 2024
043f5ed
r
cwhanse Jan 29, 2024
3e4d8f7
Update pvlib/irradiance.py
cwhanse Jan 29, 2024
2e4f2f2
Merge branch 'main' into ref_ground_diffuse
cwhanse Feb 1, 2024
816a8f8
Merge branch 'main' into ref_ground_diffuse
cwhanse Feb 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
try to get the sphinx reference
  • Loading branch information
@cwhanse
cwhanse committed Jan 26, 2024
commit 1bd9fb8ee94a5accd7e2f18f2a85f09c9b29666a
3 changes: 1 addition & 2 deletions pvlib/irradiance.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -581,12 +581,11 @@ def get_ground_diffuse(surface_tilt, ghi, albedo=.25, surface_type=None):
grounddiffuse : numeric
Ground reflected irradiance. :math:`W/m^2`


Notes
-----
Ground diffuse irradiance is the the last term of equations 3, 4, 7, 8,
10, 11, and 12 in [1]_. Table of albedo values by ``surface_type``
Copy link
Member
@adriesse adriesse Jan 28, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you describe what that last term is, rather than refer to a paper that's not openly accessible?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a preprint here but one has to find it by searching the title.

in :py:data::`SURFACE_ALBEDOS` from [2]_, [3]_, [4]_.
are from [2]_, [3]_, [4]_; see :py:data:`SURFACE_ALBEDOS`.

References
----------
Expand Down
0