search

LDeakin / zarrs

A rust library for the Zarr storage format for multidimensional arrays and metadata
Apache License 2.0
104 stars 10 forks source link

Change performance metrics and usage log storage transformers to storage adapters #77

Closed LDeakin closed 1 month ago

LDeakin commented 1 month ago

These were shoehorned in as storage transformers before storage adapters were introduced.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 69.38776% with 15 lines in your changes missing coverage. Please review.

Project coverage is 83.24%. Comparing base (cfb65d4) to head (746fb88). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...storage/src/storage_adapter/performance_metrics.rs 50.00% 14 Missing :warning:
zarrs/src/array/array_builder.rs 66.66% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #77 +/- ## ========================================== + Coverage 82.98% 83.24% +0.25% ========================================== Files 162 162 Lines 22655 22444 -211 ========================================== - Hits 18801 18683 -118 + Misses 3854 3761 -93 ```

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