Skip to content

Commit 2c1a6e0

Browse files
authored
Drop temporary workaround for get_codec (#361)
* Drop temporary workaround for `get_codec` This workaround was added because `get_codec` was modifying its argument. However as of Numcodecs 0.6.0, `get_codec` does not modify its argument as it takes a copy and modifies the copy instead. Given as we now require Numcodecs 0.6.0, this workaround is no longer needed. Hence we drop it. * Link this to the Numcodecs upgrade release note
1 parent d747f4c commit 2c1a6e0

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

docs/release.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ Bug fixes
2929

3030
* The required version of the `numcodecs <http://numcodecs.rtfd.io>`_ package has been upgraded
3131
to 0.6.2, which has enabled some code simplification and fixes a failing test involving
32-
msgpack encoding. By :user:`John Kirkham <jakirkham>`, :issue:`360`, :issue:`352`, :issue:`355`,
33-
:issue:`324`.
32+
msgpack encoding. By :user:`John Kirkham <jakirkham>`, :issue:`361`, :issue:`360`, :issue:`352`,
33+
:issue:`355`, :issue:`324`.
3434

3535
* Failing tests related to pickling/unpickling have been fixed. By :user:`Ryan Williams <ryan-williams>`,
3636
:issue:`273`, :issue:`308`.

zarr/core.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -166,9 +166,6 @@ def _load_metadata_nosync(self):
166166
if config is None:
167167
self._compressor = None
168168
else:
169-
# temporary workaround for
170-
# https://github.com/zarr-developers/numcodecs/issues/78
171-
config = dict(config)
172169
self._compressor = get_codec(config)
173170

174171
# setup filters

0 commit comments

Comments
 (0)