search

catalystneuro / datta-lab-to-nwb

MIT License
1 stars 2 forks source link

Add support for Raw Moseq-Extract output #51

Closed pauladkisson closed 1 year ago

pauladkisson commented 1 year ago

Fixes #50

pauladkisson commented 1 year ago

Code fully converts the output of moseq2-extract (See Moseq2 for a breakdown of different depth-moseq pipeline steps).

Raw moseq2-pca output (PCs) and moseq2-model output (syllables) are not supported yet and will require separate DataInterfaces to maximize reusability for the datta lab.

pauladkisson commented 1 year ago

@CodyCBakerPhD, I confirmed that there is no easy way to break this DataInterface: it expects exact types by name in the init, can't pass in some arbitrary data field, can't add extra parameters.

CodyCBakerPhD commented 1 year ago

Next steps:

1) Move to a temporally aligned interface, lines https://github.com/catalystneuro/datta-lab-to-nwb/pull/51/files#diff-89a37d11a954b4631ea551537ea0ace0e36f407a0f13f7f18a8ad9d7ca787875R84-R87 then move to the conversion script or converter protocol for temporal alignment

2) Remove unrelated metadata schema fields: https://github.com/catalystneuro/datta-lab-to-nwb/pull/51/files#diff-89a37d11a954b4631ea551537ea0ace0e36f407a0f13f7f18a8ad9d7ca787875R47-R72

CodyCBakerPhD commented 1 year ago

Also

  1. Move ndx-moseq-extract to CatalystNeuro and publish official version when stable