Skip to content

Migration for dispatchable gas plants and update scheme#1734

Open
claudiavalkenier wants to merge 1 commit intomasterfrom
add-must-run-gas-plants
Open

Migration for dispatchable gas plants and update scheme#1734
claudiavalkenier wants to merge 1 commit intomasterfrom
add-must-run-gas-plants

Conversation

@claudiavalkenier
Copy link
Copy Markdown

@claudiavalkenier claudiavalkenier commented Apr 8, 2026

Context

For a certain project, the goal is to introduce two must-run gas-fired electricity plants, based on existing dispatchable gas-fired electricity plants. These are the CCGT gas plant and the gas turbine plant.

This means that the current CCGT gas plant and gas turbine plants will be renamed to dispatchable, including all related dependencies. Subsequently, duplicates of both plant types will be created and redefined as must-run.

Initially, this will only be implemented for the future year, meaning that no additional data needs to be read from the dataset.

Implemented changes

  • Migration (for changed input names)

Related

Goes with pull requests 3433, and 4698.

Review

Functional reviewer: @louispt1

  • The following (screenshot) came up during the migration. The migration itself was successful, but it may still need to be checked?
image

Also tagging @kndehaan here!

Checklist

  • I have tested these changes
  • I have updated documentation as needed
  • I have tagged the relevant people for review

@claudiavalkenier claudiavalkenier changed the title Migration for dispatchable gas plants and update scheme (DRAFT) Migration for dispatchable gas plants and update scheme Apr 8, 2026
Copy link
Copy Markdown
Member

@louispt1 louispt1 left a comment

Choose a reason for hiding this comment

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

Looks great - the error that came up with the logging seems like something to do with the recent ruby 4 / rails 8 upgrade. Probably a few keys are not encoded properly and it wasn't showing up until now.

@louispt1 louispt1 force-pushed the add-must-run-gas-plants branch from 0719387 to 96fcce8 Compare April 8, 2026 12:36
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