Skip to content
This repository has been archived by the owner on Oct 30, 2024. It is now read-only.

Fix nat gas cap handling #29

Merged
merged 10 commits into from
Mar 21, 2023
Merged

Fix nat gas cap handling #29

merged 10 commits into from
Mar 21, 2023

Conversation

jacobvjk
Copy link
Member

@jacobvjk jacobvjk commented Mar 20, 2023

This PR:

  • replaces the natural gas cap with allowance band for the simpler 2 sided natural gas cap to reflect the recent IEA and JRC scenario pathways
  • the 2 sided cap for the bridge technology (currently only natural gas power) is split out into an optional function apply_bridge_technology_cap() to clean up calculate_company_tech_deviation() significantly
  • function argument bridge_tech gets explicitly allowed values and rlang::arg_match treatment for more robustness. the corresponding input argument checks are cleaned up
  • adds handling for the JRC scenario for SDA sectors, as the name gets slightly chopped up when running r2dii.analysis (related to separate_metric_from_name() splits up some scenario names wrong r2dii.analysis#425)

@jacobvjk jacobvjk changed the title Bug nat gas cap Fix nat gas cap handling Mar 21, 2023
@jacobvjk jacobvjk marked this pull request as ready for review March 21, 2023 10:20
Copy link
Collaborator

@MonikaFu MonikaFu left a comment

Choose a reason for hiding this comment

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

LGTM

@jacobvjk jacobvjk merged commit 5ea3430 into main Mar 21, 2023
@jacobvjk jacobvjk deleted the bug_nat_gas_cap branch March 21, 2023 10:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants