NeurodataWithoutBorders / pynwb

A Python API for working with Neurodata stored in the NWB Format
https://pynwb.readthedocs.io
Other
174 stars 85 forks source link

[Feature]: Create separate workflow for testing allensdk example #1679

Open rly opened 1 year ago

rly commented 1 year ago

What would you like to see added to PyNWB?

The allensdk has strict requirements on the versions of hdmf and pynwb as a result (currently the requirement is hdmf<=3.4.7 but the latest pynwb requires hdmf>=3.5.1). This creates a version conflict when we try to test the latest version of pynwb and hdmf - specifically when we try to test our sphinx gallery example that requires allensdk and so we install allensdk.

allensdk also has not historically supported the latest version of Python (3.10 is not supported at the moment) and allensdk has a ton of requirements that slows down builds.

Is your feature request related to a problem?

No response

What solution would you like?

I suggest we create a separate CI workflow for testing allensdk compatibility that handles the dependency issues above and so that the other gallery tests do not need to rely on an older version of Python or be slowed down in the install. This separate workflow can run nightly like the ROS3 workflow.

Do you have any interest in helping implement the feature?

Yes.

Code of Conduct

rly commented 1 year ago

Oops. This is a duplicate of https://github.com/NeurodataWithoutBorders/pynwb/issues/1659