Merge pull request #612 from CCI-Tools/jg-585-animation-error

Jg 585 animation error
CCI-Tools · Apr 19, 2018 · 5f3a17f · 5f3a17f
2 parents 6a05c25 + 8aa46e1
commit 5f3a17f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,4 +1,9 @@
-## Version 2.0.0.dev9 (in dev)
+## Version 2.0.0.dev10 (in development)
+
+* Fixed a bug with animation generation [#585](https://github.com/CCI-Tools/cate/issues/585)
+* Upgrade to using newer xarray version after an upstream bugfix [#579](https://github.com/CCI-Tools/cate/issues/579)
+
+## Version 2.0.0.dev9
 
 * Representative default variables [#590](https://github.com/CCI-Tools/cate/issues/590).
 * Tasks are no longer executed in parallel [#606](https://github.com/CCI-Tools/cate/issues/606).

diff --git a/cate/ops/animate.py b/cate/ops/animate.py
@@ -288,4 +288,4 @@ def _get_min_max(data, monitor=None):
         with monitor.child(1).observing("find maximum"):
             data_max = data.max()
 
-    return (data_min, data_max)
+    return (data_min.values, data_max.values)
diff --git a/environment.yml b/environment.yml
@@ -37,8 +37,8 @@ dependencies:
   - scipy >=0.19,<1.0
   - shapely >=1.6,<2.0
   - tornado >=5.0,<6.0
-  # xarray >= 0.10.1 has an issue with time decoding see #579
-  - xarray ==0.10.0
+  # Require >= 0.10.3 due to #579
+  - xarray >=0.10.3
   #
   # for testing only
   #