NeurodataWithoutBorders / matnwb

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

better shape constraint error message #370

Closed bendichter closed 2 years ago

bendichter commented 2 years ago

see https://github.com/NeurodataWithoutBorders/helpdesk/discussions/16

lawrence-mbf commented 2 years ago

Wouldn't this be a request for better shape validation in MatNWB? The error in the ticket looks like it comes from a pynwb validation. Was there a MatNWB when exporting the file from MATLAB?

lawrence-mbf commented 2 years ago

Sorry,

Was there a MatNWB error when exporting the file from MATLAB?

bendichter commented 2 years ago

oh yes, you are right. Do we do shape checking in MatNWB?

lawrence-mbf commented 2 years ago

I thought we did but I'm betting it's much more loose than the validators.

lawrence-mbf commented 2 years ago

I'll take a look

lawrence-mbf commented 2 years ago

We do but it appears to be too loose.