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
42 stars 38 forks source link

Optimize documentation about Earth mover distance in distance_metric preprocessor #2423

Open schlunma opened 3 months ago

schlunma commented 3 months ago

Description

The old docs are slightly misleading.

Link to documentation: https://esmvaltool--2423.org.readthedocs.build/projects/ESMValCore/en/2423/recipe/preprocessor.html#distance-metric


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 3 months ago

Codecov Report

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

Project coverage is 94.51%. Comparing base (7ce1a62) to head (c853e34).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2423 +/- ## ======================================= Coverage 94.51% 94.51% ======================================= Files 246 246 Lines 14023 14023 ======================================= Hits 13254 13254 Misses 769 769 ```

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

bouweandela commented 3 months ago

Maybe this needs a review from someone who understands?

valeriupredoi commented 3 months ago

Maybe this needs a review from someone who understands?

blog-9-banner-a6431d114beb300d12aa0ffc8af66f229b2010470b18873f7eb4e551fe1f6c46

valeriupredoi commented 1 month ago

Maybe this needs a review from someone who understands?

blog-9-banner-a6431d114beb300d12aa0ffc8af66f229b2010470b18873f7eb4e551fe1f6c46

as inspired as my comment was, I think Mr CAT is unavailable for a review, anyone else that's not busy building the new ECMWF building? :grin: