More lazy fixes and preprocessing functions

bouweandela commented 9 months ago

Description

Various small improvements in lazy data computations by using coordinate methods has_bounds(), core_points(), core_bounds() where possible.

Related to #674 and #42

Before you get started

[x] ☝ Create an issue to discuss what you are going to do

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.

[x] 🧪 The new functionality is relevant and scientifically sound
[x] 🛠 This pull request has a descriptive title and labels
[x] 🛠 Code is written according to the code quality guidelines
[x] 🛠 Unit tests have been added
[x] 🛠 Changes are backward compatible
[x] 🛠 The list of authors is up to date
[x] 🛠 All checks below this pull request were successful

To help with the number pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

codecov[bot] commented 9 months ago

Codecov Report

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

Comparison is base (6f30fa6) 93.82% compared to head (9bad950) 93.97%. Report is 5 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #2325 +/- ## ========================================== + Coverage 93.82% 93.97% +0.15% ========================================== Files 240 240 Lines 13289 13273 -16 ========================================== + Hits 12468 12473 +5 + Misses 821 800 -21 ```

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

valeriupredoi commented 8 months ago

brill work, fellas :beers:

ESMValGroup / ESMValCore