Skip to content

Update steel diffusion constraints #247

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

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

GamzeUnlu95
Copy link
Contributor

@GamzeUnlu95 GamzeUnlu95 commented Nov 13, 2024

Update steel diffusion constraints as follows:

  • removed growth_activity_lo and initial_activity_lo for bof_steel from 2030 onwards
  • changed the growth_activity_up for dri_gas_steel, dri_h2_steel, and eaf_steel to 0.075 from 2030 onwards
  • changed the initial_activity_up for dri_gas_steel and dri_h2_steel to 1.0 from 2030 onwards

How to review

No review is needed, just changing the parameters in Excel data file as agreed before.

PR checklist

  • Continuous integration checks all ✅

Copy link

codecov bot commented Nov 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.5%. Comparing base (6032244) to head (e4e8eaf).
Report is 255 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main    #247     +/-   ##
=======================================
- Coverage   76.5%   75.5%   -1.0%     
=======================================
  Files        203     203             
  Lines      15546   15546             
=======================================
- Hits       11896   11748    -148     
- Misses      3650    3798    +148     

see 7 files with indirect coverage changes

@glatterf42 glatterf42 added the material MESSAGEix-Materials variant label Nov 13, 2024
Copy link
Member

@glatterf42 glatterf42 left a comment

Choose a reason for hiding this comment

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

Thanks, this looks good to me :)
Please use either the rebase or squash strategy when merging this PR.

@GamzeUnlu95 GamzeUnlu95 merged commit a073f64 into main Nov 13, 2024
26 checks passed
@GamzeUnlu95 GamzeUnlu95 deleted the steel_diffusion_const branch November 13, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
material MESSAGEix-Materials variant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants