Skip to content

fix: test/config.electricity.yaml compatible with v25.04#1698

Merged
fneum merged 3 commits intoPyPSA:masterfrom
JuliaSvn:master
May 25, 2025
Merged

fix: test/config.electricity.yaml compatible with v25.04#1698
fneum merged 3 commits intoPyPSA:masterfrom
JuliaSvn:master

Conversation

@JuliaSvn
Copy link
Copy Markdown
Contributor

@JuliaSvn JuliaSvn commented May 20, 2025

Make it compatible with v25.04:

Removing of the plotting section (that now belongs to plotting.default.yaml) and add estimate_renewable_capacities to avoid this error:

ERROR:root:Uncaught exception
Traceback (most recent call last):
File "C:\Users\username\PypsaProject\pypsa-eur.snakemake\scripts\tmpynd5xia6.add_electricity.py", line 1256, in
attach_GEM_renewables(n, tech_map, snakemake.input)
File "C:\Users\username\PypsaProject\pypsa-eur.snakemake\scripts\tmpynd5xia6.add_electricity.py", line 889, in attach_GEM_renewables
tech_string = ", ".join(tech_map.values()) #This was the original code
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: sequence item 0: expected str instance, list found

Closes # (if applicable).

Changes proposed in this Pull Request

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 enabled auto-merge (squash) May 25, 2025 10:53
@fneum fneum merged commit b4ff1c8 into PyPSA:master May 25, 2025
12 checks passed
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