Closed heikoklein closed 3 months ago
Attention: Patch coverage is 50.00000%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 79.30%. Comparing base (
a65f838
) to head (ea7119c
). Report is 468 commits behind head on main-dev.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@AugustinMortier and @charlienegri I would like to have both your reviews, in particular with the change to using the default-range for the contours from (min-max) to (0-10).
as far as I can tell your cmap edit applies only to the case when var
is NOT in var_ranges_defaults.keys()
, so it will not affect cams2_83 (which was the scope I had edited modelmaps_engine.py
within), as all the relevant variables should be in var_ranges_defaults
... so no problems for me. but Augustin is the one best equipped to evaluate this.
Can this be merged? @charlienegri
Can this be merged? @charlienegri
ok for me, Augustin is the only one who could object but I think the changes do not impact existing projects (which should have the variables listed in var_ranges_defaults
)
Because those are new variables, and it's nor replacing current scales, there's no objection from my side :)
Change Summary
Several fixes to get map-plotting useful for the emep-reports like
Related issue number
Checklist