Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update zarr version #3598

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update zarr version #3598

wants to merge 3 commits into from

Conversation

alejoe91
Copy link
Member

@alejoe91 alejoe91 commented Jan 7, 2025

Zarr 2.18.0 used to trigger some errors. Let's see if latest releases fixed the issues.

@alejoe91 alejoe91 added the packaging Related to packaging/style label Jan 7, 2025
@alejoe91
Copy link
Member Author

alejoe91 commented Jan 7, 2025

@h-mayorquin

seems like the most recent hdmf-zarr/pynwb versions makes the use_pynwb option to fail. Not sure why honestly. The zarr files seem to be ok, and in fact when using zarr directly (use_pynwb=False) all tests pass.

Can you help me look into this? One option could be to simply enforce use_pynwb=False when a zarr backend is detected.

@zm711
Copy link
Collaborator

zm711 commented Jan 7, 2025

Maybe related to the issue at neo. pynwb calling hdmf caused our tests to fail due to changes at the hdmf level in allowable symbols in strings. They started forbidding : and / maybe due to path parsing?

@h-mayorquin
Copy link
Collaborator

Yeah, I can take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packaging Related to packaging/style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants