hdmf-dev / hdmf-zarr

Zarr I/O backend for HDMF
https://hdmf-zarr.readthedocs.io/
Other
7 stars 6 forks source link

Temp Fix for Reading S3 Bucket #238

Open mavaylon1 opened 1 day ago

mavaylon1 commented 1 day ago

Motivation

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

How to test the behavior?

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

Checklist

codecov-commenter commented 1 day ago

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 86.21%. Comparing base (fedf38e) to head (a00345d).

Files with missing lines Patch % Lines
src/hdmf_zarr/backend.py 33.33% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #238 +/- ## ========================================== - Coverage 86.36% 86.21% -0.15% ========================================== Files 5 5 Lines 1188 1190 +2 Branches 297 298 +1 ========================================== Hits 1026 1026 - Misses 103 104 +1 - Partials 59 60 +1 ```

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


🚨 Try these New Features:

mavaylon1 commented 1 day ago

Note: this is a temp fix for the benchmarks. real fix will come after export is merged