Skip to content
This repository was archived by the owner on Dec 6, 2023. It is now read-only.

Do not pre-cythonize for sdist and update build scripts #182

Merged
merged 3 commits into from
Jun 6, 2021
Merged

Do not pre-cythonize for sdist and update build scripts #182

merged 3 commits into from
Jun 6, 2021

Conversation

StrikerRUS
Copy link
Contributor

@StrikerRUS StrikerRUS commented May 28, 2021

Refer to scikit-learn/scikit-learn#14863 and scikit-learn/scikit-learn#15567.

This PR reverts this part of #108.

  • in release mode you build from .cpp files this allow to build from a released .tar.gz without cython

@StrikerRUS StrikerRUS requested review from mblondel and vene May 28, 2021 21:09
@StrikerRUS
Copy link
Contributor Author

As a next step we can automatically create wheel files for all platforms at GitHub Actions and upload them to PyPI. So for the majority of users (who pip install sklearn-contrib-lightning) nothing will be changed, they won't need to compile from sources and depend on Cython.

@mblondel
Copy link
Member

mblondel commented Jun 4, 2021

LGTM

@StrikerRUS StrikerRUS merged commit a9419ba into scikit-learn-contrib:master Jun 6, 2021
@StrikerRUS StrikerRUS deleted the build branch June 6, 2021 13:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants