Skip to content

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

Merged
jjerphan merged 15 commits into
scikit-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 into
scikit-learn:mainfrom
thomasjpfan:drop_python3_7

Conversation

@thomasjpfan

@thomasjpfan thomasjpfan commented Feb 26, 2022

Copy link
Copy Markdown
Member

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

@ogrisel ogrisel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

Comment thread pyproject.toml Outdated
@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

ogrisel commented Feb 26, 2022

Copy link
Copy Markdown
Member

@adrinjalali @lesteve merge?

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

@jeremiedbb jeremiedbb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM !

Comment thread doc/faq.rst
Comment thread setup.py Outdated

@jjerphan jjerphan left a comment

Copy link
Copy Markdown
Member

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