Skip to content

Add wind-dependent freestanding PVsyst temperature preset - #2938

Open
sahaja9707 wants to merge 1 commit into
pvlib:mainfrom
sahaja9707:codex/2834-pvsyst-wind
Open

sahaja9707 wants to merge 1 commit into
pvlib:mainfrom
sahaja9707:codex/2834-pvsyst-wind

Conversation

@sahaja9707

Copy link
Copy Markdown

The named PVsyst temperature presets currently all have u_v=0, so selecting them does not account for wind speed. Add freestanding_wind with u_c=25.0 and u_v=1.2, the PVUSA correlation described in PVsyst's heat transfer documentation.

Document the wind measurement height and the need to match coefficients to the measurement conditions. This addresses the free-standing portion of #2834; rooftop-specific coefficients are outside this change.

Validation: all 34 tests in tests/test_temperature.py pass; Flake8 on both changed Python files and git diff --check pass. A full documentation build has not been run.

AI assistance was used to prepare this patch. The coefficients and measurement guidance were checked against the primary source above, and the implementation was tested locally.

  • Addresses add free standing and rooftop thermal coeffs for nonzero windspeed #2834 (free-standing case only; does not close the full issue)
  • I am familiar with the contributing guidelines
  • I attest that all AI-generated material has been vetted for accuracy and is in compliance with the pvlib license
  • Tests added
  • Adds description and name entries in the appropriate "what's new" file, including the issue and contributor links
  • New code is fully documented, including the preset table and source reference in the existing API docstring
  • Pull request is nearly complete and ready for detailed review
  • Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue

@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown

Hey @sahaja9707! 🎉

Thanks for opening your first pull request! We appreciate your
contribution. Please ensure you have reviewed and understood the
contributing guidelines.

If AI is used for any portion of this PR, you must vet the content
for technical accuracy.

Finally, be sure the PR description includes the PR
checklist,
and complete the items you are able to.

@sahaja9707

Copy link
Copy Markdown
Author

I've made a change to add the freestanding_wind PVsyst temperature preset using the published coefficients u_c=25.0 and u_v=1.2.

The PR also documents the wind measurement height and adds a numerical test. All 34 temperature tests and style checks pass.

This addresses the free-standing portion of #2834. Could you please review the PR and merge it if everything looks good? I’m happy to address any feedback.

@cwhanse

cwhanse commented Nov 8, 2026

Copy link
Copy Markdown
Member

@sahaja9707 you're ahead of the maintainers with this one :) We need to get consensus that pvlib should change these defaults, before reviewing this PR. Thanks for your patience.

@mikofski

mikofski commented Nov 8, 2026

Copy link
Copy Markdown
Member

I support this improvement.

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.

3 participants