hed-standard / ndx-hed

HED extensions for Neurodata Without Borders (nwb)
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

HedVersionAttr serialization issue #8

Closed oruebel closed 4 months ago

oruebel commented 10 months ago

Test for the HedVersionAttr version that uses attributes to store the hed version are currently failing because the created object seems to be recognized as LabMetaData but not as HedVersionAttr. See failing asserts in the tests here: https://github.com/hed-standard/ndx-hed/blob/43113e34aa867b6fa8c20a93e37c2581b0c7fb75/src/pynwb/tests/test_hed_version_attr.py#L78-L82

VisLab commented 4 months ago

The adding of the parameter to DynamicTable to allow extra parameters seems to have addressed this issue --- serialization now works.