sdss / lvmdrp

Local Volume Mapper (LVM) Data Reduction Pipeline
BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

Test numpy > 2 and resolve any issues #116

Open havok2063 opened 3 months ago

havok2063 commented 3 months ago

We should test the DRP with numpy > 2.0. numpy > 2 introduced some breaking changes, and changes in algorithms, which may result in numerical differences. Additionally, not all packages have added support yet for numpy > 2.

To resolve some package dependencies, you need

When testing, I see some failures between pandas and numpy, with the error ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject. Even though pandas == 2.2.2 is installed, there may still be some incompatibilities with either pandas, h5py, or some internal c-code. See, e.g. https://github.com/numpy/numpy/issues/26710 or https://stackoverflow.com/questions/78634235/numpy-dtype-size-changed-may-indicate-binary-incompatibility-expected-96-from

numpy pinned to < 2 in https://github.com/sdss/lvmdrp/pull/115. Also see https://numpy.org/doc/2.0/numpy_2_0_migration_guide.html and https://numpy.org/doc/2.0/release/2.0.0-notes.html