Skip to content

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

Merged
jjerphan merged 15 commits intoscikit-learn:mainfrom
thomasjpfan:drop_python3_7
Feb 28, 2022
Merged

MNT Drops Python 3.7 in CI, wheel building, and docs#22617
jjerphan merged 15 commits intoscikit-learn:mainfrom
thomasjpfan:drop_python3_7

Conversation

@thomasjpfan
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member Author

The pypy error fix is at #22619

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 <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

Development

Successfully merging this pull request may close these issues.

4 participants