Update scenario used

[MonikaFu]

Update the thresholds and default scenario to align with what will be used in Swiss COP 2024.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 9.69%. Comparing base (2d3f1ca) to head (7c67ff4).

Files	Patch %	Lines
R/prep_scores.R	0.00%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main    #325   +/-   ##
=====================================
  Coverage   9.69%   9.69%           
=====================================
  Files         27      27           
  Lines       2187    2187           
=====================================
  Hits         212     212           
  Misses      1975    1975           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[AlexAxthelm]

@MonikaFu It looks like the extdata directory is big for this package. Is that expected?

https://github.com/RMI-PACTA/pacta.executive.summary/actions/runs/9226981271/job/25387941392?pr=325#step:6:291

[cjyetman]

Looks like the documentation needs to be regenerated to avoid the Warning in R CMD check

[jdhoffa]

I hope it's ok @MonikaFu I took the liberty to rebuild the docs since you are on PTO.
Checks should pass now

[jdhoffa]

I see that the codecob/patch action is failing, though I remember we decided somewhere else that this was an action we might want to deactivate. @cjyetman can you remind me how/ where you disabled that portion of the codecov action?

See #326

[jdhoffa]

@MonikaFu I think we will need to update this file as well, do you have any idea how to do this?
https://github.com/RMI-PACTA/pacta.executive.summary/blob/main/data-raw/remaining_carbon_budgets.R

Edit: Antoine is currently working on this. Tracked by #318

[cjyetman]

I see that the codecob/patch action is failing, though I remember we decided somewhere else that this was an action we might want to deactivate. @cjyetman can you remind me how/ where you disabled that portion of the codecov action?

See #326

e.g. RMI-PACTA/pacta.portfolio.import#94

[MonikaFu]

@AlexAxthelm extdata contains all the Rmd templates and example images so yeah, to be expected I'd say. It is not ideal that the templates are part of this package probably. Going forward (if we are planning to generate executive summaries regularly) we should probably move them to a separate directory.

[MonikaFu]

Thanks @jdhoffa for taking care of the documentation! My bad. I will open a separate PR to remove the codecov action for now. In general since the PR is approved by @jdhoffa and @AlexAxthelm are we good to go to merge it now?

[jdhoffa]

Good to merge!