Skip to content

Fix/issue 30 sector config - #191

Open
omari91 wants to merge 6 commits into
PyPSA:mainfrom
omari91:fix/issue-30-sector-config
Open

Fix/issue 30 sector config#191
omari91 wants to merge 6 commits into
PyPSA:mainfrom
omari91:fix/issue-30-sector-config

Conversation

@omari91

@omari91 omari91 commented Sep 7, 2026

Copy link
Copy Markdown

Before asking for a review for this PR make sure to complete the following checklist:

Changes proposed in this Pull Request

  • Fixes Issue Configurations of sectorparameter get overwritten #30 (Configurations of sector parameter get overwritten): Changed the initialisation logic in scripts/pypsa-de/build_scenarios.py to use not in membership tests instead of .get(...) truthiness checks for the sector, industry, and solving config dictionaries. This prevents custom configurations from being completely overwritten and erased when the user passes an empty dictionary or False.

Checklist

Required:

  • Changes are tested locally and behave as expected.
  • Code and workflow changes are documented.
  • A brief description of the changes has been added to Changelog.md. (Please add if needed!)

If applicable:

  • Changes in configuration options are reflected in scripts/lib/validation. (N/A)
  • For new data sources or versions, these instructions have been followed. (N/A)
  • New rules are documented in the appropriate doc/*.rst files. (N/A)

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