search

NeurodataWithoutBorders / matnwb

A Matlab interface for reading and writing NWB files
BSD 2-Clause "Simplified" License
49 stars 32 forks source link

Trial length not checked #139

Closed owoolnough closed 5 years ago

owoolnough commented 5 years ago

Found a potential issue in types.core.TimeIntervals that could do with a warning/check. Using this to set trial details will accept trial definitions with unequal lengths e.g.:

trials = types.core.TimeIntervals( ... ... 'start_time', types.core.VectorData('data', [1:1000],'description','start time of trial'),... 'stop_time', types.core.VectorData('data', [1:500],'description','end time of trial'),... );

This will export fine but will not load trials out when imported

lawrence-mbf commented 5 years ago

Since MatNWB has no internal knowledge of the intended DynamicTable Structure, it may be recommended to use #111 when that gets completed.

nclack commented 5 years ago

Closing in favor of #111 which will address this issue.