pynwb icon indicating copy to clipboard operation
pynwb copied to clipboard

[Feature]: Warn to use index argument on call to add_trial (or add_row)

Open CodyCBakerPhD opened this issue 11 months ago • 1 comments

What would you like to see added to PyNWB?

From https://github.com/NeurodataWithoutBorders/helpdesk/discussions/71#discussioncomment-8638454

User was confused by HDMF level error during io.write. Caused by not declaring a column to be 'indexed' and giving it ragged input across different rows (which worked fine to add to the in-memory NWBFile object)

Is your feature request related to a problem?

No response

What solution would you like?

Warn or error on trying to add the value while still in-memory rather than waiting until writing to disk

Do you have any interest in helping implement the feature?

No.

Code of Conduct

CodyCBakerPhD avatar Mar 01 '24 16:03 CodyCBakerPhD

@stephprince can you help with this issue? I believe this is will most likely need to be addressed in HDMF so we may want to transfer the issue to HDMF.

oruebel avatar Mar 01 '24 17:03 oruebel

This was fixed in hdmf (https://github.com/hdmf-dev/hdmf/pull/1066) and will be available at the next release.

stephprince avatar Mar 19 '24 18:03 stephprince