search

catalystneuro / mease-lab-to-nwb

MIT License
3 stars 2 forks source link

Add multi accelerometer #18

Closed CodyCBakerPhD closed 3 years ago

CodyCBakerPhD commented 3 years ago

@bendichter @alejoe91 Extended the conversion script and accelerometer part to handle multiple recordings.

Wound up being a lot less code to just combine the different extractors into one basic inheritance from the MultiRecordingTimeExtractor, each member of it being an intan type. Still works great on folders containing a single file.

bendichter commented 3 years ago

what's the new package, crc32c?

CodyCBakerPhD commented 3 years ago

Not 100% sure right now; it prompted me to install it at some point today, about the same time as edlio and sonpy, but it might be for some other branch. This branch appears to run fine without it, so it's removed.

bendichter commented 3 years ago

ok, I created a PR on eldio to add requirements. Once that gets merged and released we won't need to list this ourselves anymore, but let's keep it here for the time being