ChangLabUcsf / ndx-electrical-stim

An NWB extension for storing electrical stimulation waveforms and metadata for neural stimulation.
Other
2 stars 1 forks source link

Do we need to define a target table for StimTable's bipolar_pair column? #6

Open jessierliu opened 4 years ago

jessierliu commented 4 years ago

@bendichter

We discussed how PyNWB's Units class has a column called electrodes that can reference the electrodes table. Those column entries are arrays, so they need a target table defined. For the StimTable, the analogous column is bipolar_pair, which are DynamicTableRegion pointers to a created bipolar scheme table.

So this column's entries already have a target table, since DynamicTableRegion.table exists. Do we still need to define a table in some way as is done in Units?

https://github.com/NeurodataWithoutBorders/pynwb/blob/dev/src/pynwb/misc.py#L195-L209