Skip to content

Fix wildcard adjustment options#1817

Merged
fneum merged 1 commit intoPyPSA:masterfrom
koen-vg:wildcard-opts-fix
Aug 26, 2025
Merged

Fix wildcard adjustment options#1817
fneum merged 1 commit intoPyPSA:masterfrom
koen-vg:wildcard-opts-fix

Conversation

@koen-vg
Copy link
Copy Markdown
Contributor

@koen-vg koen-vg commented Aug 26, 2025

The adjustments config section has gotten an updated schema a while ago, and the {opts} and {sector_opts} wildcards (in particular, the carrier+{c|p|m} options) were never updated to respect this new schema.

I guess these wildcard options haven't been used in a little while since nobody caught this earlier!

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.

The `adjustments` config section has gotten an updated schema a while
ago, and the `{opts}` and `{sector_opts}` wildcards (in particular,
the `carrier+{c|p|m}` options) were never updated to respect this new
schema.
@fneum fneum merged commit a1032ea into PyPSA:master Aug 26, 2025
8 checks passed
measrainsey pushed a commit to open-energy-transition/pypsa-wal that referenced this pull request Sep 5, 2025
The `adjustments` config section has gotten an updated schema a while
ago, and the `{opts}` and `{sector_opts}` wildcards (in particular,
the `carrier+{c|p|m}` options) were never updated to respect this new
schema.
toniseibold pushed a commit that referenced this pull request Dec 1, 2025
The `adjustments` config section has gotten an updated schema a while
ago, and the `{opts}` and `{sector_opts}` wildcards (in particular,
the `carrier+{c|p|m}` options) were never updated to respect this new
schema.
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