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

DO NOT MERGE tests for pv panel #49

Draft
wants to merge 5 commits into
base: production
Choose a base branch
from
Draft

Conversation

MaaJoo13
Copy link
Collaborator

@MaaJoo13 MaaJoo13 commented Oct 25, 2024

Summary of the discussion

DO NOT MERGE

tests for pv_panel facade, results can be found in examples/scripts/test_pv.xlsx

Two options are valid:

  • Link pv_panel capacity to the source capacity with a Constraint, so pv_panel capacity will be in m²
  • this is only problematic if the power output of the modules installed on 1m² is higher than 1kW (not likely, but maybe possible...please test for a location close to the equator where panels have small tilt and are close to each other as well as high irradiation values)
  • keep pv_capacity openly expandable and it will automatically be the maximum power output of the pv_panel within the provided time series
  • this maximum power output is usually lower than the real peak capacity
  • the real peak capacity can be accessed from the facade, please look at the PR#51 from feature/pv

Please note that all bad configurations from test_pv_noload_1 (cut off/scaled down profiles) should be applicable to test_pv_noload_8 with the final pv-panel-8 facade. I just didnt test all of them.
Volatile (evtl. with excess to be safe) or dispatchable with profile can both be used as a source.
They can both be fixed or optimized up to capacity_potential.

Type of change (CHANGELOG.md)

Added

  • Add a new class (#)

Changed

  • Update a definition (#)

Removed

  • Remove a broken link (#)

Workflow checklist

Automation

Closes #

PR-Assignee

Reviewer

  • 🐙 Follow the Reviewer Guidelines
  • 🐙 Provided feedback and show sufficient appreciation for the work done

@Bachibouzouk Bachibouzouk changed the title tests for pv panel DO NOT MERGE tests for pv panel Oct 29, 2024
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.

1 participant