pynwb icon indicating copy to clipboard operation
pynwb copied to clipboard

extend AnnotationSeries class to incorporate other variables

Open luiztauffer opened this issue 5 years ago • 2 comments

2) Feature Request

When storing annotations in the AnnotationSeries class, besides the text the only available info to add is the time. It would be useful to be able to add user-defined info, e.g. channel number, for each annotation. Perhaps it would be a good idea to make it a DynamicTable?

Problem/Use Case

There are cases when the annotations might be specific to other variables.

Checklist

  • [x] Have you ensured the feature or change was not already reported ?
  • [x] Have you included a brief and descriptive title?
  • [x] Have you included a clear description of the problem you are trying to solve?
  • [ ] Have you included a minimal code snippet that reproduces the issue you are encountering?
  • [x] Have you checked our Contributing document?

luiztauffer avatar May 22 '19 14:05 luiztauffer

@luiztauffer thanks for the suggestion. We'll need to take a closer at AnnotationSeries. In the meantime, if you need a DynamicTable to annotate time intervals an alternative may be for you to simply add your own TimeInterval table with NWBFile.create_time_intervals (or NWBFile.add_time_intervals).

oruebel avatar May 22 '19 16:05 oruebel

@oruebel thanks for the tip! I'll look into that

luiztauffer avatar May 22 '19 16:05 luiztauffer