ESMValGroup / ESMValTool

ESMValTool: A community diagnostic and performance metrics tool for routine evaluation of Earth system models in CMIP
https://www.esmvaltool.org
Apache License 2.0
226 stars 128 forks source link

Change example correlation diagnostic so it uses a more meaningful statistic #625

Closed bouweandela closed 5 months ago

bouweandela commented 6 years ago

So we do not forget the discussion in #596, this code https://github.com/ESMValGroup/ESMValTool/blob/60a89f7828025c599615bcb5932b1917a40fb333/esmvaltool/diag_scripts/examples/correlate.py#L48

should probably be updated so it uses either: scipy.stats.mstats.ks_twosamp scipy.stats.ks_2samp or this: scipy.stats.anderson_ksamp as some people seem a bit critical about the KS test.

valeriupredoi commented 6 years ago

assigned myself on this one, my intention is to start looking into developing a serious statistical module for ESMValTool, this is a good starting point

RCHG commented 6 years ago

Regarding this issue/enhancement here there is information that I hope will be useful.

bouweandela commented 6 years ago

We will also support R diagnostics in the near future, see https://github.com/ESMValGroup/ESMValTool/pull/631, so no need to use rpy2.

bouweandela commented 5 months ago

Feel free to re-open if anyone has plans to do this.