search

AMLab-Amsterdam / lie_learn

Computations involving Lie groups and harmonic analysis
MIT License
190 stars 47 forks source link

Support numpy>2.0 #30

Closed kalekundert closed 1 month ago

kalekundert commented 2 months ago

As of numpy==2.0, the cython np.int_t type no longer exists. Instead, an explicitly-sized type such as np.int32_t or np.int64_t must be used. Here's a relevant stack overflow post:

https://stackoverflow.com/questions/21851985/difference-between-np-int-np-int-int-and-np-int-t-in-cython

lie_learn currently uses np.int_t in one place, and as a result cannot be installed with up-to-date numpy. The PR replaces this one usage with np.int64_t, which should mean the exact same thing.

kalekundert commented 1 month ago

@tscohen I just updated this PR to fix the merge conflict from #31.