I was benchmarking different compression options and realized BitRound was overwriting my array in place.

import shutil
import dask.array as da
from numcodecs import BitRound

def write_to_store(array, **kwargs):
    shutil.rmtree("test.zarr", ignore_errors=True)
    array.to_zarr("test.zarr", **kwargs)
    print(array.mean().compute())

array = da.random.default_rng(42).random((1,)).persist()

write_to_store(array)
write_to_store(array, filters=[BitRound(keepbits=5)])
write_to_store(array)

0.9167441575549085
0.921875
0.921875

Looks like there was discussion on this in the original PR but nothing came of it. Making an explicit copy seems to work fine as well, not sure which is preferred?

cc @rabernat

TODO:

• Unit tests and/or doctests in docstrings
• Tests pass locally
• Docstrings and API docs for any new/modified user-facing classes and functions
• Changes documented in docs/release.rst
• Docs build locally
• GitHub Actions CI passes
• Test coverage to 100% (Codecov passes)