Summary
Several DynamicTableRegion columns across the core schema carry no table attribute specifying which table they reference. By contrast, the icephys relational tables (SimultaneousRecordingsTable, SequentialRecordingsTable, etc.) do declare typed table constraints via a dtype refspec on the table attribute. This inconsistency means the schema cannot validate that these region columns point to the correct table.
Affected fields
| File |
Type |
Field |
Suggested target type |
nwb.ecephys.yaml |
ElectricalSeries |
electrodes |
ElectrodesTable |
nwb.ecephys.yaml |
FeatureExtraction |
electrodes |
ElectrodesTable |
nwb.misc.yaml |
Units |
electrodes |
ElectrodesTable |
nwb.misc.yaml |
DecompositionSeries |
source_channels |
ElectrodesTable ?? |
nwb.ophys.yaml |
RoiResponseSeries |
rois |
PlaneSegmentation |
Current icephys pattern (for reference)
The icephys relational tables use a typed table attribute with a dtype refspec to enforce the target table type. This is the pattern that should be adopted here for consistency.
Open questions
- Does
source_channels in DecompositionSeries have a well-defined target table to constrain to?
Summary
Several
DynamicTableRegioncolumns across the core schema carry notableattribute specifying which table they reference. By contrast, the icephys relational tables (SimultaneousRecordingsTable,SequentialRecordingsTable, etc.) do declare typed table constraints via a dtype refspec on thetableattribute. This inconsistency means the schema cannot validate that these region columns point to the correct table.Affected fields
nwb.ecephys.yamlElectricalSerieselectrodesElectrodesTablenwb.ecephys.yamlFeatureExtractionelectrodesElectrodesTablenwb.misc.yamlUnitselectrodesElectrodesTablenwb.misc.yamlDecompositionSeriessource_channelsElectrodesTable??nwb.ophys.yamlRoiResponseSeriesroisPlaneSegmentationCurrent icephys pattern (for reference)
The icephys relational tables use a typed
tableattribute with a dtype refspec to enforce the target table type. This is the pattern that should be adopted here for consistency.Open questions
source_channelsinDecompositionSerieshave a well-defined target table to constrain to?