The Wayback Machine - https://web.archive.org/web/20220918054317/https://github.com/scikit-learn/scikit-learn/pull/22674
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

MAINT Bump versions to match oldest-supported-numpy #22674

Merged

Conversation

thomasjpfan
Copy link
Member

@thomasjpfan thomasjpfan commented Mar 3, 2022

Reference Issues/PRs

Follow up to #22617

What does this implement/fix? Explain your changes.

This PR bumps the NumPy minimal dependency to match the ones seen in oldest-supported-numpy

CC @ogrisel

@jeremiedbb
Copy link
Member

jeremiedbb commented Mar 4, 2022

I think it's worth checking what backward compat tool is not necessary anymore when bumping the min version of the dependencies. Otherwise we end up keeping dead code in the code base for a long time. I think this bump only allows to remove

if np_version < parse_version("1.17"):

mprofile_20220302134119.dat Outdated Show resolved Hide resolved
ogrisel
ogrisel approved these changes Mar 4, 2022
Copy link
Member

@ogrisel ogrisel left a comment

LGTM!

@ogrisel
Copy link
Member

ogrisel commented Mar 4, 2022

I did a bunch of quick git greps and could not find other occurrences of #22674 (comment).

@ogrisel ogrisel merged commit 269bdb9 into scikit-learn:main Mar 4, 2022
28 of 29 checks passed
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.

None yet

4 participants