Skip to content

Split geological CO2 storage into onshore and offshore. Use new storage potentials#2296

Draft
dklein-pik wants to merge 19 commits intoremindmodel:developfrom
dklein-pik:uptake
Draft

Split geological CO2 storage into onshore and offshore. Use new storage potentials#2296
dklein-pik wants to merge 19 commits intoremindmodel:developfrom
dklein-pik:uptake

Conversation

@dklein-pik
Copy link
Contributor

@dklein-pik dklein-pik commented Feb 24, 2026

Purpose of this PR

Split geological CO2 storage into onshore and offshore. Use new storage potentials

Type of change

Parts concerned

  • ☑️ GAMS Code
  • ◻️ R-scripts
  • ◻️ Documentation (GAMS incode documentation, comments, tutorials)
  • ☑️ Input data / CES parameters
  • ◻️ Tests, CI/CD (continuous integration/deployment)
  • ☑️ Configuration (switches in main.gms, default.cfg, and scenario_config*.csv files)
  • ◻️ Other (please give a description)

Impact

  • ◻️ Bug fix
  • ◻️ Refactoring
  • ☑️ New feature
  • ☑️ Change of parameter values or input data (including CES parameters)
  • ◻️ Minor change (default scenarios show only small differences)
  • ◻️ Fundamental change of results of default scenarios

Checklist

  • I executed the automated model tests (make test) after my final commit and all tests pass (FAIL 0)
  • I adjusted the reporting in remind2 if and where it was needed
  • I adjusted the madrat packages (mrremind and other packages involved) for input data generation if and where it was needed
  • My code follows the coding etiquette
  • I explained my changes within the PR, particularly in hard-to-understand areas
  • I checked that the in-code documentation is up-to-date
  • I adjusted forbiddenColumnNames in readCheckScenarioConfig.R in case the PR leads to deprecated switches
  • I updated the CHANGELOG.md correctly (added, changed, fixed, removed, input data/calibration)

Requires new input data f_geoStorPot.cs3r from mrremind::calcPotentialGeologicalStorage
…grade anyway but more technologies to come (ccsinjeon and ccsinjeoff), each with its own storage potential defined by pm_dataccs.
Reduce dimensions of q_emiCdrAll to calculate the sum of ccsinjeon and ccsinjeoff and not individually
@dklein-pik dklein-pik requested a review from amerfort February 25, 2026 11:12
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.

1 participant