hdmf-dev / hdmf-zarr

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

[Feature]: support Python 3.12 #166

Closed bendichter closed 4 months ago

bendichter commented 5 months ago

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

We are trying to enhance NeuroConv to support Python 3.12 (https://github.com/catalystneuro/neuroconv/pull/626), however we are having trouble due to the numcodecs dependency. Since numcodecs is a dependency of this package, and this package does not yet test for Python 3.12 compatibility, so I think it makes sense to work on support for Python 3.12 first.

Is your feature request related to a problem?

No response

What solution would you like?

The first step would involve changing the workflows to move the Python version upper bound from 3.11 to 3.12. Then we can see what is failing and go from there.

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct

h-mayorquin commented 5 months ago

Related: https://github.com/hdmf-dev/hdmf-zarr/issues/162

oruebel commented 5 months ago

however we are having trouble due to the numcodecs dependency.

The issue with the numcodecs version should be resolved by https://github.com/hdmf-dev/hdmf-zarr/pull/163 , which is part of the latest 0.6.0 release. Hopefully that helps you move forward on the NeuroConv side.

The first step would involve changing the workflows to move the Python version upper bound from 3.11 to 3.12. Then we can see what is failing and go from there

@mavaylon1 can you take a stab at this

CodyCBakerPhD commented 5 months ago

NeuroConv is all good now: https://github.com/catalystneuro/neuroconv/pull/626

Still a good idea for y'all to extend your CI though

mavaylon1 commented 5 months ago

Will do.