Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

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

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

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

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

@thomasjpfan thomasjpfan marked this pull request as ready for review February 26, 2022 03:09
@thomasjpfan
Copy link
Member Author

The pypy error fix is at #22619

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@thomasjpfan
Copy link
Member Author

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 commented Feb 26, 2022

@adrinjalali @lesteve merge?

@ogrisel ogrisel added the Quick Review For PRs that are quick to review label Feb 26, 2022
Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM !

Copy link
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after addressing remaining comments, thank you @thomasjpfan.

@jjerphan jjerphan merged commit f1d3417 into scikit-learn:main Feb 28, 2022
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this pull request 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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants