search

catalystneuro / datta-lab-to-nwb

MIT License
1 stars 2 forks source link

[Follow-up] Behavior syllables should use ndx-events #13

Closed CodyCBakerPhD closed 1 year ago

CodyCBakerPhD commented 1 year ago

Especially given our revelation in the last meeting that we can use the paper figure to map the top 37 syllables using human-readable labels, this implies that the better representation of the neurodata object BehavioralSyllable (just a simple TimeSeries contained in a BehavioralTimeSeries named SyllableTimeSeries) is actually to have it be a LabeledEvent using the ndx-events extension

Now, I suspect it was originally modeled in the TimeSeries fashion because of compatibility with the figure reproductions (which could end up eventually in a notebook attached to the DANDI set); do you think the new representation would unduly complicate that aspect @pauladkisson?

pauladkisson commented 1 year ago

I agree, ndx-events should make sense for this data. The TimeSeries representation was mostly a convenience choice, but it shouldn't be too hard to convert back to a TimeSeries-style representation for the figures.