Closed oruebel closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 86.08%. Comparing base (
4c9af4a
) to head (e28904e
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@mavaylon1 this is ready for review.
Motivation
For benchmarking and when consolidated metadata may be bad, it will be useful to force reading without consolidated metadata. This is needed for the nwb-benchmarks.
This PR:
r-
to force that a file is opened inr
mode without using consolidated metadata.ZarrIO.__open_file_consolidated
used properties ofZarrIO
instead of the provided input parameters. This did not cause any errors so far, since the values matched but is still something that could have caused issues later on.-r
read modeZarrStoreTestCase
which assumed in thetearDown
that the output file was always being created, so clean-up would fail if a test-case did not create a file.Checklist
ruff
from the source directory.