hdmf-dev / hdmf-zarr

Zarr I/O backend for HDMF
https://hdmf-zarr.readthedocs.io/
Other
7 stars 5 forks source link

[Feature]: NWBZarrIO should have load_namespaces=True by default #198

Open rly opened 1 month ago

rly commented 1 month ago

What would you like to see added to HDMF-ZARR?

Following this change in PyNWB https://github.com/NeurodataWithoutBorders/pynwb/pull/1748

Is your feature request related to a problem?

No response

What solution would you like?

Consistency between NWBHDF5IO and NWBZarrIO

Do you have any interest in helping implement the feature?

No.

Code of Conduct

mavaylon1 commented 1 month ago

I'll take a look next week.

mavaylon1 commented 2 days ago

Context: We want to have load_namespaces to be True by default. This reads cached schemas and automatically tries to generate the API for any missing namespaces. The prime example being extensions.

On the pynwb side, I do not see any tests involving a roundtrip for a file with extensions, testing the whole load_namespace finding the cached schemas.