Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Correct infinite_sheds view factor from row to sky and ground; expose vf functions in bifacial.utils #1666

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

Merged
merged 27 commits into from
Jun 23, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
f684623
correct view factor from row to sky and ground
mikofski Feb 15, 2023
ef346dd
update latex math in vf_row_gnd
mikofski Feb 15, 2023
510e8ad
revise vf_row_sky and test
Mar 22, 2023
fdc7a63
remake tests for vf_row_ground
cwhanse Mar 23, 2023
784a977
fix my mistakes
cwhanse Mar 23, 2023
c48b35c
fix row_ground_integ
cwhanse Mar 23, 2023
253bb89
new vf functions in utils
cwhanse Mar 24, 2023
2a9bff1
merge 0.9.5 upstream
cwhanse Mar 24, 2023
5837fd3
stickler
cwhanse Mar 24, 2023
3246196
use functions from utils
cwhanse Mar 24, 2023
a0451b7
documents
cwhanse Mar 24, 2023
a77dbb0
revise whatsnew
cwhanse Mar 24, 2023
09f3f86
add technical document
cwhanse Mar 24, 2023
e7b7d5f
doc edits
cwhanse Apr 3, 2023
fc6c7d9
fix conflicts
cwhanse May 15, 2023
df56b3a
Merge branch 'fix_vf_row_sky_gh1665' of https://github.com/mikofski/p…
cwhanse May 15, 2023
6a026ad
move fns to utils
cwhanse May 15, 2023
47bd4de
more fcn shuffling
cwhanse May 15, 2023
7e1be57
update docs
cwhanse May 15, 2023
02bc783
reorder arguments
cwhanse May 16, 2023
fbf3d93
docstring edits
cwhanse May 16, 2023
bdb80ea
add defaults to _integ functions
cwhanse May 16, 2023
ccac4b2
Merge branch 'main' into fix_vf_row_sky_gh1665
cwhanse May 24, 2023
710a5bd
Merge branch 'main' of https://github.com/pvlib/pvlib-python into fix…
cwhanse Jun 7, 2023
667e3f5
Merge branch 'fix_vf_row_sky_gh1665' of https://github.com/mikofski/p…
cwhanse Jun 7, 2023
a11257d
Merge remote-tracking branch 'upstream/main' into pr/1666
kandersolar Jun 23, 2023
b6996e5
fix whatsnew formatting issues
kandersolar Jun 23, 2023
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
fix conflicts
  • Loading branch information
cwhanse committed May 15, 2023
commit fc6c7d94d696b5a466fc92e261010ac125fe1d41
7 changes: 7 additions & 0 deletions 7 docs/sphinx/source/whatsnew/v0.9.6.rst
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,8 @@ Bug fixes
* Corrects an error in view factor calculations which are part of
:py:func:`pvlib.bifacial.infinite_sheds.get_irradiance` (:issue:`1665`, :pull:`1666`)
cwhanse marked this conversation as resolved.
Show resolved Hide resolved

* `data` can no longer be left unspecified in
:py:meth:`pvlib.modelchain.ModelChain.run_model_from_effective_irradiance`. (:issue:`1713`, :pull:`1720`)

Testing
~~~~~~~
Expand All @@ -41,4 +43,9 @@ Contributors
~~~~~~~~~~~~
* Mark Mikofski (:ghuser:`mikofski`)
* Cliff Hansen (:ghuser:`cwhanse`)
* Lakshya Garg (:ghuser:`Lakshyadevelops`)
* Adam R. Jensen (:ghuser:`adamrjensen`)
* Siddharth Kaul (:ghuser:`k10blogger`)
* Kshitiz Gupta (:ghuser:`kshitiz305`)
* Stefan de Lange (:ghuser:`langestefan`)

36 changes: 19 additions & 17 deletions 36 pvlib/bifacial/infinite_sheds.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,9 +95,22 @@ def _poa_ground_shadows(poa_ground, f_gnd_beam, df, vf_gnd_sky):


def _poa_sky_diffuse_pv(dhi, gcr, surface_tilt):
"""
Sky diffuse POA from averaged view factors combined for both shaded and
unshaded parts of the surface.
r"""
Integrated view factors from the shaded and unshaded parts of
the row slant height to the sky.

Parameters
----------
f_x : numeric
Fraction of row slant height from the bottom that is shaded from
direct irradiance. [unitless]
surface_tilt : numeric
Surface tilt angle in degrees from horizontal, e.g., surface facing up
= 0, surface facing horizon = 90. [degree]
gcr : float
Ratio of row slant length to row spacing (pitch). [unitless]
npoints : int, default 100
Number of points for integration. [unitless]

A detailed calculation would be

Expand All @@ -114,6 +127,9 @@ def _poa_sky_diffuse_pv(dhi, gcr, surface_tilt):

Parameters
----------
f_x : numeric
Fraction of row slant height from the bottom that is shaded from
direct irradiance. [unitless]
dhi : numeric
Diffuse horizontal irradiance (DHI). [W/m^2]
gcr : float
Expand Down Expand Up @@ -178,20 +194,6 @@ def _poa_ground_pv(poa_ground, gcr, surface_tilt):
Reduce ground-reflected irradiance to account for limited view of the
ground from the row surface.

A detailed calculation would be

poa_ground *
(f_x * vf_shade_ground_integ + (1 - f_x) * vf_noshade_ground_integ)

where vf_shade_ground_integ is the average view factor between 0 and f_x
(the shaded portion). But the average view factor is

1/(f_x - 0) Integral_0^f_x vf(x) dx

so the detailed calculation is equivalent to

poa_ground * 1/(1 - 0) Integral_0^1 vf(x) dx

Parameters
----------
poa_ground : numeric
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.