What would you like to see added to the NWBInspector?
We were in the process of making something like this inspired by the previous commented code, but was also discovered in a recent manual inspection.
Basically, an entire row or column of a DynamicTable, or an entire axis of a TimeSeries, should not all be NaN.
There are some exceptions that we have to observe such as x,y,z for ElectrodeTable. Basically any field that is required schematically but unknown in practice, we'd have to skip. But for axes optionally added or certainly custom fields, they should not be included if they are unknown.
Pseudocode or other logic of the check
all(np.isnan(getattr(obj, "data")))
Do you have any interest in helping implement the check function?
Yes.
Code of Conduct
What would you like to see added to the NWBInspector?
We were in the process of making something like this inspired by the previous commented code, but was also discovered in a recent manual inspection.
Basically, an entire row or column of a DynamicTable, or an entire axis of a TimeSeries, should not all be NaN.
There are some exceptions that we have to observe such as
x,y,zfor ElectrodeTable. Basically any field that is required schematically but unknown in practice, we'd have to skip. But for axes optionally added or certainly custom fields, they should not be included if they are unknown.Pseudocode or other logic of the check
all(np.isnan(getattr(obj, "data")))Do you have any interest in helping implement the check function?
Yes.
Code of Conduct