search

PyWavelets / pywt

PyWavelets - Wavelet Transforms in Python
http://pywavelets.readthedocs.org
MIT License
1.97k stars 460 forks source link

CI: limit contourpy to <1.1, to avoid building win32 from source #680

Closed rgommers closed 10 months ago

rgommers commented 10 months ago

As seen in gh-679, contourpy fails to build from source on 32-bit. This is fixable, it should build from source just fine. But we don't want to deal with that if we can avoid it. And contourpy 1.0.7 works just fine.

[skip actions]

rgommers commented 10 months ago

Some Cython 3.0 warnings, but those are not yet fixable, since there isn't a good replacement for this syntax:

  warning: pywt\_extensions\config.pxi:3:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
  warning: pywt\_extensions\_pywt.pxd:15:0: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
  warning: pywt\_extensions\config.pxi:3:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
rgommers commented 10 months ago

That worked, it installs contourpy-1.0.7-cp38-cp38-win32.whl for 32-bit Python, rather than trying to build 1.1.0 from source.

These are also straightforward fixes, so I'll merge this as well to keep the ball rolling.