catalystneuro / datta-lab-to-nwb

MIT License
1 stars 2 forks source link

Reorganize conversion to support different experiment_types #38

Closed pauladkisson closed 1 year ago

pauladkisson commented 1 year ago

Fixes #37. Also, adds BaseDattaInterface to handle all non-specific metadata. Note, this PR is branched off of #36 --> should wait for #36 to merge.

CodyCBakerPhD commented 1 year ago

Overall looks fine, a couple suggestions for improved documentation and operability

CodyCBakerPhD commented 1 year ago

@pauladkisson LGTM, just needs a conflict resolution