Skip to content

feat: add recalculate_modeling_correction_factor to PVSystem#30

Merged
javlor merged 2 commits into
dnv-opensource:mainfrom
timvanbenthem-dnv:tvb/recalculate-mcf
Jul 9, 2026
Merged

feat: add recalculate_modeling_correction_factor to PVSystem#30
javlor merged 2 commits into
dnv-opensource:mainfrom
timvanbenthem-dnv:tvb/recalculate-mcf

Conversation

@timvanbenthem-dnv

Copy link
Copy Markdown
Contributor

recalculate_modeling_correction_factor always defaults to True when using the PVSystem model. Added a feature to set the recalculate_modeling_correction_factor when constructing a PVSystem class object.

@javlor

javlor commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Thank you for the contribution @timvanbenthem-dnv. The new attribute is relevant and correctly applied. Please just review the formatting of pvsystem.py with Ruff lint and format check. See here how it can be used locally. Note that coding agents are good with this too.

@javlor javlor merged commit ea30108 into dnv-opensource:main Jul 9, 2026
14 checks passed
@timvanbenthem-dnv timvanbenthem-dnv deleted the tvb/recalculate-mcf branch July 9, 2026 08:15
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.

2 participants