intake/intake-xarray

Update requirements to use msgpack (not msgpack-python)

Closed this issue · 4 comments

Currently intake-xarray lists msgpack-python as a requirement. However upstream renamed this to msgpack. Would be good to have the requirement updated here accordingly

intake-xarray/setup.py

Lines 12 to 13 in aceb731

INSTALL_REQUIRES = ['intake >=0.5.2', 'xarray >=0.17.0', 'zarr', 'dask >=2.2', 'netcdf4', 'fsspec>0.8.3',
'msgpack-python', 'requests']

cc @jrbourbeau @martindurant @weiji14

OK, so what does this do with the pypi version, do I need to make a new one?

Probably easiest to do the msgpack-python to msgpack rename, tag a new version for intake-xarray and push a new PyPI release. The alternative (yanking the intake-xarray v0.4.2 release from PyPI and republishing) is much harder, and probably not a good idea.

Oh, and we should probably tag the release as v0.5.0 because the minimum version of Python was bumped from Python 3.6 to Python 3.7 (due to a minimum xarray version of 0.17.0, up from 0.12.0) in #101. As per NEP29:

Minimum Python and NumPy version support should be adjusted upward on every major and minor release, but never on a patch release.

fine, done