-
Notifications
You must be signed in to change notification settings - Fork 84
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 Zarr IO tutorial #1834
Add Zarr IO tutorial #1834
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1834 +/- ##
=======================================
Coverage 91.99% 91.99%
=======================================
Files 27 27
Lines 2623 2623
Branches 685 685
=======================================
Hits 2413 2413
Misses 138 138
Partials 72 72
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
docs/gallery/advanced_io/zarr_io.py
Outdated
# For more information, see the :hdmf-zarr:`hdmf-zarr documentation<>`. | ||
|
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.
I would make this a callout .. note::
and also point to the NWB specific tutorials in addition to the general hdmf-zarr docs.
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.
Thanks for adding this tutorial. I added a couple of minor suggestions, but overall this looks good to me.
Motivation
fix #1832
Checklist
flake8
from the source directory.