-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add NWBZarrIO.read_nwb convenience function #226
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #226 +/- ##
==========================================
+ Coverage 85.56% 85.64% +0.07%
==========================================
Files 5 5
Lines 1185 1198 +13
Branches 292 297 +5
==========================================
+ Hits 1014 1026 +12
+ Misses 115 113 -2
- Partials 56 59 +3 ☔ View full report in Codecov by Sentry. |
🆗 |
@@ -12,6 +12,7 @@ | |||
* Added test for opening file with consolidated metadata from DANDI. @mavaylon1 [#206](https://github.com/hdmf-dev/hdmf-zarr/pull/206) | |||
* Add dimension labels compatible with xarray. @mavaylon1 [#207](https://github.com/hdmf-dev/hdmf-zarr/pull/207) | |||
* Added link_data --> clear_cache relationship to support repacking zarr nwbfiles: [#215](https://github.com/hdmf-dev/hdmf-zarr/pull/215) | |||
* Added `NWBZarrIO.read_nwb` convenience method to simplify reading an NWB file. @oruebel [#226](https://github.com/hdmf-dev/hdmf-zarr/pull/226) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mavaylon1, was this released already? it is on the changelog on the 0.9.0 release but the PR was merged a week ago. Is that a changelog error?
Motivation
Fix #225
TODO
NWBZarrIO.read_nwb
read_nwb
read_nwb
read_nwb
to the tutorialsChecklist
ruff
from the source directory.