From 423b44d319259d1078c3dd377e0d178ad6209455 Mon Sep 17 00:00:00 2001 From: Kai Pak Date: Sat, 3 Mar 2018 16:37:48 -0500 Subject: [PATCH 1/2] Check for minimum Zarr version. ws --- xarray/backends/zarr.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/xarray/backends/zarr.py b/xarray/backends/zarr.py index b0323b51f17..9f8973d6aa3 100644 --- a/xarray/backends/zarr.py +++ b/xarray/backends/zarr.py @@ -2,6 +2,7 @@ from base64 import b64encode from itertools import product +from distutils.version import LooseVersion import numpy as np @@ -271,6 +272,14 @@ class ZarrStore(AbstractWritableDataStore): def open_group(cls, store, mode='r', synchronizer=None, group=None, writer=None): import zarr + min_zarr = '2.2' + + if LooseVersion(zarr.__version__) < min_zarr: + raise NotImplementedError("Zarr version %s or greater is " + "required by xarray. See zarr " + "installation " + "http://zarr.readthedocs.io/en/stable/" + "#installation" % min_zarr) zarr_group = zarr.open_group(store=store, mode=mode, synchronizer=synchronizer, path=group) return cls(zarr_group, writer=writer) From 9ae7bce99ce33555d2e40e24e9c0ef5047eeef8f Mon Sep 17 00:00:00 2001 From: Kai Pak Date: Sun, 4 Mar 2018 23:02:06 -0500 Subject: [PATCH 2/2] Exclude version check from coverage. additional ws. --- xarray/backends/zarr.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xarray/backends/zarr.py b/xarray/backends/zarr.py index 9f8973d6aa3..769334992ae 100644 --- a/xarray/backends/zarr.py +++ b/xarray/backends/zarr.py @@ -274,7 +274,7 @@ def open_group(cls, store, mode='r', synchronizer=None, group=None, import zarr min_zarr = '2.2' - if LooseVersion(zarr.__version__) < min_zarr: + if LooseVersion(zarr.__version__) < min_zarr: # pragma: no cover raise NotImplementedError("Zarr version %s or greater is " "required by xarray. See zarr " "installation "