Skip to content
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

Pv tilt update #578

Merged
merged 6 commits into from
Apr 24, 2024
Merged

Pv tilt update #578

merged 6 commits into from
Apr 24, 2024

Conversation

bpulluta
Copy link
Collaborator

@bpulluta bpulluta commented Apr 19, 2024

Please check if the PR fulfills these requirements

  • CHANGELOG.md is updated
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

(Bug fix, feature, docs update, ...)

default update for 'PV tilt'.

  • In core/pv.jl a change was made to make sure we are using the same assumptions as PVWatts guidelines, the default tilt angle for a fixed array should be 20 degrees, irrespective of it being a rooftop (0) or ground-mounted (open-rack)(1) system. By default the tilt will be set to 20 degrees for fixed ground-mount and rooftop, and 0 degrees for axis-tracking (array_type = (2), (3), or (4))

"The PVWatts® default value for the tilt angle depends on the array type: For a fixed array, the default value is 20 degrees, and for one-axis tracking the default value is zero. A common rule of thumb for fixed arrays is to set the tilt angle to the latitude of the system's location to maximize the system's total electrical output over the year. Use a lower tilt angle favor peak production in the summer months when the sun is high in the sky, or a higher tilt angle to increase output during winter months. Higher tilt angles tend to cost more for racking and mounting hardware, and may increase the risk of wind damage to the array."

What is the current behavior?

(You can also link to an open issue here)
The current default PV tilt angle for a rooftop is set to 10 degrees and for fixed ground-mount to 20 degrees.

What is the new behavior (if this is a feature change)?

The new behavior sets the tilt angle for fixed PV rooftop and ground-mount to 20 degrees and for axis-tracking to 0 degrees. This aligns it with the pvwatts defaults.

Does this PR introduce a breaking change?

(What changes might users need to make in their application due to this PR?)
Specify a PV tilt angle if they may need to use a lower tilt angle for their fixed arrays than the new default of 20 due to hardware cost, wind, or other consideration consideration

Other information:

@bpulluta bpulluta requested a review from adfarth April 19, 2024 20:57
Copy link
Collaborator

@Bill-Becker Bill-Becker left a comment

Choose a reason for hiding this comment

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

Nice work! Just a suggestion to remove some excess description from the Changelog.

CHANGELOG.md Outdated Show resolved Hide resolved
@bpulluta
Copy link
Collaborator Author

@adfarth, seems like all the test are passing now. Please let me know if its okay, once you've had a change to review. Thank you!

Copy link
Collaborator

@adfarth adfarth left a comment

Choose a reason for hiding this comment

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

Look great!

@bpulluta bpulluta merged commit 7b04800 into develop Apr 24, 2024
@bpulluta bpulluta deleted the pv_tilt_update branch April 24, 2024 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants