Closed jdhoffa closed 4 months ago
Attention: Patch coverage is 0%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 9.69%. Comparing base (
cb14a97
) to head (6fdbb8c
).
Files | Patch % | Lines |
---|---|---|
R/render_executive_summary.R | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
It turns out that, while many of the
prep_
functions do utilize a parameterizedscenario
argument, we had just been relying on the default1.5C-Unif
in most cases, for most functions.This PR does a few things, all broadly related to parameterizing the
scenario
andscenario_source
selection:render_executive_summary
gains argumentscenario_source_selected
2024
templates gain specification forscenario_selected
andscenario_source_selected
(as applicable)GECO2021
and1.5C-Unif
) for backward compatibility. (I don't know how necessary that really is, but hey ho, why not follow good practices)