Commit a1f3f6c6 authored by Thibault Hallouin's avatar Thibault Hallouin
Browse files

update to latest evalhyd version

Version which replaces `xt::stddev` with a custom made one. Thus was
done to fix a bug with rtensors, but it seems to have also fixed #1:
the calculation of KGE/KGEPRIME now works on 1D pytensors.
No related merge requests found
Pipeline #38172 passed with stage
in 1 minute and 43 seconds
Showing with 6 additions and 8 deletions
+6 -8
Subproject commit 02f0889222bd18d12ee1417bcd6069fe71749aa6 Subproject commit 90924065cc6365f65e3b33214884eb352a01e913
...@@ -48,13 +48,11 @@ class TestMetrics(unittest.TestCase): ...@@ -48,13 +48,11 @@ class TestMetrics(unittest.TestCase):
def test_metrics_1d(self): def test_metrics_1d(self):
for metric in self.expected.keys(): for metric in self.expected.keys():
# TODO: fix failure for KGE and KGEPRIME with 1D pytensors with self.subTest(metric=metric):
if metric not in ('KGE', 'KGEPRIME'): numpy.testing.assert_almost_equal(
with self.subTest(metric=metric): evalhyd.evald(_obs[0], _prd[0], [metric])[0],
numpy.testing.assert_almost_equal( self.expected[metric][0]
evalhyd.evald(_obs[0], _prd[0], [metric])[0], )
self.expected[metric][0]
)
class TestTransform(unittest.TestCase): class TestTransform(unittest.TestCase):
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment