ESMValGroup / ESMValCore

ESMValCore: A community tool for pre-processing data from Earth system models in CMIP and running analysis scripts.
https://www.esmvaltool.org
Apache License 2.0
40 stars 36 forks source link

More lazy fixes #2450

Closed bouweandela closed 2 weeks ago

bouweandela commented 1 month ago

Description

A few more updates to avoid realizing data when applying fixes.

Related to #674.


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the ๐Ÿ›  Technical or ๐Ÿงช Scientific review.


To help with the number pull requests:

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.61%. Comparing base (ee16a95) to head (eebfc2f). Report is 6 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2450 +/- ## ========================================== - Coverage 94.62% 94.61% -0.01% ========================================== Files 246 246 Lines 14055 14045 -10 ========================================== - Hits 13299 13289 -10 Misses 756 756 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

schlunma commented 2 weeks ago

Apparently a line is too long now, could you fix that? Then I will merge