Skip to content

Validation#651

Merged
fneum merged 133 commits intomasterfrom
validation
Aug 6, 2023
Merged

Validation#651
fneum merged 133 commits intomasterfrom
validation

Conversation

@FabianHofmann
Copy link
Copy Markdown
Contributor

@FabianHofmann FabianHofmann commented Apr 25, 2023

closes #363
closes #462
closes #523

ToDos

  • Rename config.validation.yaml in Snakefile to config.yaml
  • Re-enable shipping raster
  • simple hydro constraint (p_min_pu and/or p_max_pu (per country))
  • start up cost update
  • check the efficiencies of polish coal fleet
  • crossborder code (direction, orders of magnitude)
  • solve electricity prices
  • historical dispatch as a outer constraint for nuclear
  • add a section to documentation

FabianHofmann and others added 30 commits April 25, 2023 14:44
@FabianHofmann FabianHofmann marked this pull request as ready for review August 1, 2023 07:40
@FabianHofmann FabianHofmann requested a review from fneum August 1, 2023 10:24
Copy link
Copy Markdown
Member

@fneum fneum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite a few nice new features! I have a couple of suggestions:

  • are sources for added data files listed in documentation?

@FabianHofmann
Copy link
Copy Markdown
Contributor Author

Thank you @fneum for the detailed review! @virio-andreyana will take over most of the points

@fneum
Copy link
Copy Markdown
Member

fneum commented Aug 5, 2023

Once CI goes through, good to go.

@fneum fneum enabled auto-merge August 6, 2023 07:22
@fneum fneum merged commit e34cec4 into master Aug 6, 2023
@fneum fneum deleted the validation branch August 6, 2023 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Document existing and planned validation Add rsync utility snakemake rules Validation of the Power System Model for 2019

6 participants