search

catalystneuro / mease-lab-to-nwb

MIT License
3 stars 2 forks source link

Add accelerometer #9

Closed CodyCBakerPhD closed 3 years ago

CodyCBakerPhD commented 3 years ago

@bendichter @alejoe91 It won't let me create this PR as a draft or I would; this is work still in progress but mostly done. A couple things left to add, mostly written as 'TODO's in comments.

It's fine for this particular dataset, but in a more general setting it might be nice to have the ability to specify a channel_conversions like option similar to that in ElectricalSeries (granted it wouldn't generally be for 'channels' per se but still some sort of indexed-based specification anyhow) - shall I put in a feature request for that on the NWB schema?

It doesn't currently support a lazy (or even non-lazy) stub_test option, so testing this on the 'Latest Syntalos' prototype takes a tad bit of time to complete (done within a couple minutes or less, though).

bendichter commented 3 years ago

@CodyCBakerPhD could you take a look at these merge conflicts?

CodyCBakerPhD commented 3 years ago

@bendichter Conflicts resolved, also updated new interface to nwb-conversion-tools v0.6.1