Closed jwodder closed 4 months ago
NB: asking in https://github.com/dandi/dandi-cli/pull/1371/files#r1420705289 on either we could/should reuse a higher level constructs in zarr_checksum
Removal of the Zarr code from dandischema will require rewriting the code for extracting total size from a Zarr checksum. This will mean adding zarr_checksum
as a dependency to dandischema, but:
zarr_checksum
currently requires Pydantic v2, which dandischema
does not yet support
zarr_checksum
depends on two heavy libraries that could easily be removed from the default dependencies but haven't yet: https://github.com/dandi/zarr_checksum/pull/39
Now that we have
zarr_checksum
, having a separate module for checksumming Zarrs in dandischema is redundant.dandischema.digests.zarr
withzarr_checksum
dandi/files/zarr.py
dandiapi/zarr/tests/test_zarr.py
dandischema/models.py
dandischema.digests.zarr
and its test code