-
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
Improve Zarr tutorial #1835
Improve Zarr tutorial #1835
Conversation
* link to numcodecs module * add thumbnail * clean up language a bit
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1835 +/- ##
=======================================
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. |
Co-authored-by: Ryan Ly <[email protected]>
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.
Looks good to me. You may want to crop the empty space at the top of thumbnail, but that's just nitpicking ;-)
The broken external link is unrelated -- it looks like one of the scicrunch resolvers might be down. We can adjust the URL if it's not back up in a few days.. |
No description provided.