hdmf-dev / hdmf

The Hierarchical Data Modeling Framework
http://hdmf.readthedocs.io
Other
46 stars 24 forks source link

Zarr append for datasets (non-reference) #1136

Closed mavaylon1 closed 1 week ago

mavaylon1 commented 1 week ago

Motivation

What was the reasoning behind this change? Please explain the changes briefly.

This supports appending to zarr arrays. This does not cover the edge case for dataset of references. This has the base tests for append_data. There are more tests (TBD) in hdmf-zarr that incorporate the full IO process.

How to test the behavior?

Show how to reproduce the new behavior (can be a bug fix or a new feature)

Checklist

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.56%. Comparing base (7426275) to head (57a0981).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #1136 +/- ## ========================================== - Coverage 88.79% 85.56% -3.23% ========================================== Files 45 45 Lines 9774 9778 +4 Branches 2776 2777 +1 ========================================== - Hits 8679 8367 -312 - Misses 778 1075 +297 - Partials 317 336 +19 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.