DASDAE / dascore

A python library for distributed fiber optic sensing
Other
71 stars 16 forks source link

Silixa h5 #394

Closed d-chambers closed 2 months ago

d-chambers commented 2 months ago

Description

This PR adds support for Silixa's HDF5 format, which has similar fields to the TDMS format. It also migrates prodml from pytables to h5py and removes all pytables get_format because they can cause segfaults and issue many warnings about certain types of HDF elements.

I also added a few tests to ensure the FiberIOManager returns each format exactly once and a registry for (name, version) to help ensure each FiberIO instances is not registered more than once.

Checklist

I have (if applicable):

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.83%. Comparing base (dea01ab) to head (1ca11e4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #394 +/- ## ======================================= Coverage 99.83% 99.83% ======================================= Files 106 109 +3 Lines 8625 8713 +88 ======================================= + Hits 8611 8699 +88 Misses 14 14 ``` | [Flag](https://app.codecov.io/gh/DASDAE/dascore/pull/394/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DASDAE) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/DASDAE/dascore/pull/394/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DASDAE) | `99.83% <100.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DASDAE#carryforward-flags-in-the-pull-request-comment) to find out more.

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