Skip to content

Validate configuration settingf for temporal clustering to prevent invalid setups#1882

Merged
fneum merged 3 commits intomasterfrom
inustrial_demand_factor
Nov 5, 2025
Merged

Validate configuration settingf for temporal clustering to prevent invalid setups#1882
fneum merged 3 commits intomasterfrom
inustrial_demand_factor

Conversation

@martacki
Copy link
Copy Markdown
Member

@martacki martacki commented Nov 4, 2025

Changes proposed in this Pull Request

Raise an error if temporal clustering settings are used incorrectly to avoid underestimating power demands in the sector network

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

@fneum fneum merged commit 14be760 into master Nov 5, 2025
3 of 5 checks passed
@fneum fneum deleted the inustrial_demand_factor branch November 5, 2025 14:11
toniseibold pushed a commit that referenced this pull request Dec 1, 2025
…valid setups (#1882)

* update scaling factor for industrial demands with snapshot_weightings

* remove multiplication with sn weightings, raise error instead for invalid configurations

* Update scripts/time_aggregation.py

---------

Co-authored-by: Fabian Neumann <[email protected]>
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