html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/issues/1954#issuecomment-370155913,https://api.github.com/repos/pydata/xarray/issues/1954,370155913,MDEyOklzc3VlQ29tbWVudDM3MDE1NTkxMw==,9453271,2018-03-03T15:31:24Z,2018-03-03T15:31:24Z,CONTRIBUTOR,"Thanks guys, that fixed the problem.
I could do a PR for it. Even if it 2.2 is coming out so soon, it would be nice to get a clear error for users so they don't lose a bunch of cycles trying to debug. As you noted, it would be relatively easy to implement.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370044335,https://api.github.com/repos/pydata/xarray/issues/1954,370044335,MDEyOklzc3VlQ29tbWVudDM3MDA0NDMzNQ==,1197350,2018-03-02T20:32:59Z,2018-03-02T20:32:59Z,MEMBER,"I think you're right. Since zarr 2.2 is being released so soon, it's probably silly to add such a check.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370043488,https://api.github.com/repos/pydata/xarray/issues/1954,370043488,MDEyOklzc3VlQ29tbWVudDM3MDA0MzQ4OA==,2443309,2018-03-02T20:29:49Z,2018-03-02T20:29:49Z,MEMBER,"Okay, this is just a little messy right now because we're ahead of zarr in their release schedule. We expect that will come out on Monday (see https://github.com/zarr-developers/zarr/issues/242).
Typically, we have stayed away from checking versions of all optional dependencies on import this may be one time that would be useful. The problem is these checks often fall out of date end up causing unnecessary clutter down the line.
To resolve this we can add a version check (as you've shown above) or we can wait until Monday. I'm also not opposed to mentioning in the documentation what minimum version we support is. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370042719,https://api.github.com/repos/pydata/xarray/issues/1954,370042719,MDEyOklzc3VlQ29tbWVudDM3MDA0MjcxOQ==,1197350,2018-03-02T20:26:24Z,2018-03-02T20:26:48Z,MEMBER,"@kaipak it would be great if you could put together a quick PR that would raise an error on importing zarr if the version is too old.
You would insert this around here:
https://github.com/pydata/xarray/blob/master/xarray/backends/zarr.py#L273
The code would looks something like:
```python
from distutils.version import LooseVersion
...
import zarr
if LooseVersion(zarr.__version__) < '2.2':
raise NotImplementedError(""Zarr version 2.2 or greater is required by xarray. ""
""See zarr installation instructions at http://zarr.readthedocs.io/en/stable/#installation"")","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370041032,https://api.github.com/repos/pydata/xarray/issues/1954,370041032,MDEyOklzc3VlQ29tbWVudDM3MDA0MTAzMg==,1197350,2018-03-02T20:19:40Z,2018-03-02T20:19:40Z,MEMBER,I guess we should be raising an error if zarr is too old rather than failing in this way.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370040272,https://api.github.com/repos/pydata/xarray/issues/1954,370040272,MDEyOklzc3VlQ29tbWVudDM3MDA0MDI3Mg==,1217238,2018-03-02T20:16:32Z,2018-03-02T20:16:32Z,MEMBER,"We should document this more clearly, but you need zarr 2.2 (currently in release candidate stage) for use with xarray.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370039149,https://api.github.com/repos/pydata/xarray/issues/1954,370039149,MDEyOklzc3VlQ29tbWVudDM3MDAzOTE0OQ==,9453271,2018-03-02T20:11:33Z,2018-03-02T20:11:33Z,CONTRIBUTOR,"
```
INSTALLED VERSIONS
------------------
commit: None
python: 3.6.2.final.0
python-bits: 64
OS: Linux
OS-release: 3.10.0-693.17.1.el7.centos.plus.x86_64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
xarray: 0.10.1
pandas: 0.20.3
numpy: 1.13.1
scipy: 0.19.1
netCDF4: 1.2.4
h5netcdf: 0.5.0
h5py: 2.7.0
Nio: None
zarr: 2.1.4
bottleneck: 1.2.1
cyordereddict: None
dask: 0.15.2
distributed: 1.18.1
matplotlib: 2.0.2
cartopy: None
seaborn: 0.8.0
setuptools: 36.4.0
pip: 9.0.1
conda: None
pytest: None
IPython: 6.1.0
sphinx: None
```
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370037927,https://api.github.com/repos/pydata/xarray/issues/1954,370037927,MDEyOklzc3VlQ29tbWVudDM3MDAzNzkyNw==,1197350,2018-03-02T20:06:38Z,2018-03-02T20:06:38Z,MEMBER,Can you show the output of ``xr.show_versions()``?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608