You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The end goal is to be able to do pip install git+https://github.com/letmaik/[email protected] without installing numpy and cython in advance. This is mostly for Linux and will also work on macOS.
setuptools got more clever regarding cython: https://github.com/pypa/setuptools/blob/main/CHANGES.rst#180 But see also pypa/setuptools#1317.
However, this is not enough, as there's also this bit in setup.py: include_dirs += [numpy.get_include()]. I'm not sure yet how that can be deferred. Probably using pyproject.toml to declare build dependencies. The issue with that is that the numpy version should depend on the Python version to be as ABI compatible as possible, see also the GitHub Actions workflow. Maybe the requirements specifiers allow to express this. EDIT: There's https://pypi.org/project/oldest-supported-numpy/ which seems to be useful for this.
The text was updated successfully, but these errors were encountered:
Related to #147.
The end goal is to be able to do
pip install git+https://github.com/letmaik/[email protected]
without installing numpy and cython in advance. This is mostly for Linux and will also work on macOS.setuptools got more clever regarding cython: https://github.com/pypa/setuptools/blob/main/CHANGES.rst#180 But see also pypa/setuptools#1317.
However, this is not enough, as there's also this bit in setup.py:
include_dirs += [numpy.get_include()]
. I'm not sure yet how that can be deferred. Probably usingpyproject.toml
to declare build dependencies. The issue with that is that the numpy version should depend on the Python version to be as ABI compatible as possible, see also the GitHub Actions workflow. Maybe the requirements specifiers allow to express this. EDIT: There's https://pypi.org/project/oldest-supported-numpy/ which seems to be useful for this.The text was updated successfully, but these errors were encountered: