Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add BECCS qtty target to regipol #1336

Merged
merged 2 commits into from
Jun 19, 2023
Merged

Conversation

orichters
Copy link
Contributor

@orichters orichters commented Jun 15, 2023

Purpose of this PR

  • expand the implicit quantity target calculation in regipol such that CCS and BECCS can be differentiated

  • You can enter your target in Mt CO2 in cm_implicitQttyTarget

  • Renato told me that adding too many time-steps in the switch may lead to strange behavior and bad convergence, but dragging down at both ends with values a bit below the actual goals worked very well.

  • the run below is with 2060.GLO.tax.t.CCS.biomass 4300, 2080.GLO.tax.t.CCS.biomass 4300, and it nailed it quite well, I think
    image

  • the run below is with 2060.GLO.tax.t.CCS.biomass 3800, 2080.GLO.tax.t.CCS.biomass 3800 and that also looks very convincing.
    image

Type of change

  • New feature

Checklist:

  • My code follows the coding etiquette
  • I performed a self-review of my own code
  • 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 did not adjust the reporting in remind2 as it was not necessary
  • All automated model tests pass (FAIL 0 in the output of make test)

@orichters orichters marked this pull request as ready for review June 16, 2023 07:25
@orichters
Copy link
Contributor Author

Implemented the change you requested, compiled and it looks good.

@orichters orichters merged commit fb5f2a8 into remindmodel:develop Jun 19, 2023
@orichters orichters deleted the 47BECCS branch December 6, 2023 14:06
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