Skip to content

Conversation

toniseibold
Copy link
Contributor

wrongfully set comma leads to an error in prepare_sector_network when config[sector][methanol][methanol_to_power][ccgt] is activated

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

  • Workflow with target rule ariadne_all completes without errors
  • The logic of export_ariadne_variables has been adapted to the changes
  • One or several figures that validate the changes in the PR have been posted as a comment
  • A brief description of the changes has been added to Changelog.md
  • The latest main has been merged into the PR
  • The config has a new prefix of the format YYYYMMDDdescriptive_title

toniseibold and others added 2 commits January 16, 2025 13:35
wrongfully set comma leads to an error in `prepare_sector_network` when `config[sector][methanol][methanol_to_power][ccgt]` is activated
Copy link
Contributor

github-actions bot commented Jan 16, 2025

Validator Report

I am the Validator. Download all artifacts here.
I'll be back and edit this comment for each new commit.

❗ Run failed!

Download 'logs' artifact to see more details.

  • main failed in: plot_hydrogen_network, export_ariadne_variables
  • toniseibold-bugfix passed! ✅

Model Metrics

Benchmarks Image not available Image not available Image not available

Comparing toniseibold-bugfix (02ea802) with main (c1e59d1).
Branch is 2 commits ahead and 0 commits behind.
Last updated on 2025-01-16 14:32:25 CET.

@lkstrp lkstrp merged commit ca2d8ff into main Jan 20, 2025
5 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