You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think it is bad practice to use a DynamicTable in this way because it's not a table, and it doesn't specify or validate data types. We discussed in our meeting that it may be better to provide an easy path for users to extend LabMetaData with simple, well-documented attributes (no groups, datasets, or links) for this use case. In the meantime, I think we should amend the above text to discourage use of DynamicTable in this way.
The text was updated successfully, but these errors were encountered:
From @h-mayorquin:
ChatGPT got the idea that a
DynamicTable
should be used to add extended metadata, even if the table has a single row.This might have arisen from the note in https://nwb-overview.readthedocs.io/en/latest/extensions_tutorial/extension_examples/labmetadata_extension.html#approach
which is not super clear.
I think it is bad practice to use a
DynamicTable
in this way because it's not a table, and it doesn't specify or validate data types. We discussed in our meeting that it may be better to provide an easy path for users to extend LabMetaData with simple, well-documented attributes (no groups, datasets, or links) for this use case. In the meantime, I think we should amend the above text to discourage use ofDynamicTable
in this way.The text was updated successfully, but these errors were encountered: