LorenFrankLab / rec_to_nwb

Data Migration REC -> NWB 2.0 Service
Other
2 stars 8 forks source link

Skip MCU_IO binaries and alert user #47

Closed acomrie closed 1 year ago

acomrie commented 1 year ago

We currently do not use MCU_IO data in the lab, but it is possible to extract these empty data streams from .rec files. If they have been extracted during preprocessing, we should not mix them up with our standard ECU DIOs. They are named similarly - for ex, ending in "MCU_Din3.dat" versus ending in just "Din3.dat" - so future version of rec_to_nwb should appropriately find MCU_IOs and incorporate those data in nwb files. In the interm, this fix enables us to skip MCU_IOs, alert the user, and look only for ECU DIOs here. Without this, MCU_IO binaries can incorrectly get matched up to ECU DIO names, causing errors in the data inserted in the nwb file. This fix enables finding only the ECU DIO data and matching them to the appropriate ECU DIO names. The change has been tested locally.