Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MNT Drops Python 3.7 in CI, wheel building, and docs #22617

Merged
merged 15 commits into from Feb 28, 2022

Conversation

thomasjpfan
Copy link
Member

@thomasjpfan thomasjpfan commented Feb 26, 2022

Reference Issues/PRs

Related to #21460
Related to #22614

What does this implement/fix? Explain your changes.

This PR bumps the Python version in the CI and docs.

  1. Bumps SciPy to 1.3.2 (Released Nov 2019) as it is the oldest version that supports Python 3.8.
  2. Bumps NumPy to 1.6.6 as this is the oldest version supported by conda on the defaults channel that supports Python 3.8

@thomasjpfan
Copy link
Member Author

@thomasjpfan thomasjpfan commented Feb 26, 2022

PyPy3.8 is not fully ready yet: conda-forge/conda-forge-pinning-feedstock#2089

For now, I am keeping 3.7 on just for PyPy test to run and build.

@thomasjpfan
Copy link
Member Author

@thomasjpfan thomasjpfan commented Feb 26, 2022

Looks like everything works now. The pypy build failure is the same on main.

@thomasjpfan thomasjpfan marked this pull request as ready for review Feb 26, 2022
@thomasjpfan
Copy link
Member Author

@thomasjpfan thomasjpfan commented Feb 26, 2022

The pypy error fix is at #22619

Copy link
Member

@ogrisel ogrisel left a comment

LGTM!

pyproject.toml Outdated Show resolved Hide resolved
@thomasjpfan
Copy link
Member Author

@thomasjpfan thomasjpfan commented Feb 26, 2022

Thanks for updating the pyproject.toml.

Looks like the wheel builds work. The random window build errors are expected: #22310

@ogrisel
Copy link
Member

@ogrisel ogrisel commented Feb 26, 2022

@ogrisel ogrisel added the Quick Review label Feb 26, 2022
Copy link
Member

@jeremiedbb jeremiedbb left a comment

LGTM !

doc/faq.rst Show resolved Hide resolved
setup.py Outdated Show resolved Hide resolved
Copy link
Member

@jjerphan jjerphan left a comment

LGTM after addressing remaining comments, thank you @thomasjpfan.

@jjerphan jjerphan merged commit f1d3417 into scikit-learn:main Feb 28, 2022
29 checks passed
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this issue Mar 1, 2022
)

* MNT Drops Python 3.7

* MNT Bump NumPy and SciPy

* FIX Fix build

* FIX Bump versions improved

* DOC Fixes numpy version [pypy]

* BLD [pypy] [icc-build]

* Update docs

* MAINT use scipy.optimize.LinearConstraint in test

* MAINT scipy 1.1.0 related code clean-up

* scipy>=1.3.2 in pyproject.toml's build deps

* [cd build]

* DOC Adds comment about pypy

* MAINT remove _astype_copy_false

* FIX Update check for python version in setup.py

Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
Diwakar-Gupta added a commit to Diwakar-Gupta/scikit-learn that referenced this issue Mar 7, 2022
)

* MNT Drops Python 3.7

* MNT Bump NumPy and SciPy

* FIX Fix build

* FIX Bump versions improved

* DOC Fixes numpy version [pypy]

* BLD [pypy] [icc-build]

* Update docs

* MAINT use scipy.optimize.LinearConstraint in test

* MAINT scipy 1.1.0 related code clean-up

* scipy>=1.3.2 in pyproject.toml's build deps

* [cd build]

* DOC Adds comment about pypy

* MAINT remove _astype_copy_false

* FIX Update check for python version in setup.py

Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants