8000 BLD Sets minimal supported numpy version in pyproject.toml by BenjaminLiuPenrose · Pull Request #16823 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

BLD Sets minimal supported numpy version in pyproject.toml #16823

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Apr 4, 2020
Merged

BLD Sets minimal supported numpy version in pyproject.toml #16823

merged 5 commits into from
Apr 4, 2020

Conversation

BenjaminLiuPenrose
Copy link
Contributor

Reference Issues/PRs

Fixes #16788

What does this implement/fix? Explain your changes.

SciPy>=1.2.2 are built with OpenBLAS
SciPy 1.2 requires oldest-supported-numpy (https://pypi.org/project/oldest-supported-numpy/)
numpy>=1.16.0 starts supporting AIX machine

Any other comments?

For now changes in pyproject.toml files only, other changes in relevant files will be required.

@BenjaminLiuPenrose BenjaminLiuPenrose mentioned this pull request Apr 1, 2020
Closed
rth
rth requested changes Apr 1, 2020
View reviewed changes
Copy link
Member
@rth rth left a comment

Choose a reason for hiding this comment

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

Thanks @BenjaminLiuPenrose ! A few comments below.

rth
rth approved these changes Apr 1, 2020
View reviewed changes
Copy link
Member
@rth rth left a comment

Choose a reason for hiding this comment

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

LGTM, assuming CI passes, thanks for your contribution!

(We need to wait for a second reviewer to merge).

@BenjaminLiuPenrose
Copy link
Contributor Author
BenjaminLiuPenrose commented Apr 1, 2020 •
edited
Loading

thanks

Will make an update when I know what to do with the scipy part

@BenjaminLiuPenrose
Copy link
Contributor Author

LGTM, assuming CI passes, thanks for your contribution!

(We need to wait for a second reviewer to merge).

Based on this reply [https://mail.python.org/pipermail/scipy-dev/2020-April/024115.html], no need to specify scipy version for different python version at this moment. scipy>=0.19.1 will work as usual

@cmarmo cmarmo modified the milestone: 0.11 Apr 3, 2020
@BenjaminLiuPenrose
Copy link
Contributor Author

Any one can point me out why pylatest_pip_openblas_pandas always failed in recent test?

@thomasjpfan
Copy link
Member

There is a temporary fix in #16843 for the CI.

thomasjpfan
thomasjpfan approved these changes Apr 4, 2020
View reviewed changes
Copy link
Member
@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

I see this is your first time contributing, welcome @BenjaminBossan !

LGTM

@thomasjpfan thomasjpfan changed the title minimal supported numpy and scipy version in pyproject.toml BLD Sets minimal supported numpy version in pyproject.toml Apr 4, 2020
@thomasjpfan thomasjpfan merged commit 395cfd7 into scikit-learn:master Apr 4, 2020
@BenjaminLiuPenrose
Copy link
Contributor Author

Thanks

@BenjaminLiuPenrose BenjaminLiuPenrose deleted the my_change branch April 5, 2020 02:22
gio8tisu pushed a commit to gio8tisu/scikit-learn that referenced this pull request May 15, 2020
@BenjaminLiuPenrose @gio8tisu
BLD Sets minimal supported numpy version in pyproject.toml (scikit-le…
5f4e8fd 
…arn#16823)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rth rth rth approved these changes

@thomasjpfan thomasjpfan thomasjpfan approved these changes

Assignees
No one assigned
Labels
module:linear_model
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Specify minimal supported numpy and scipy version in pyproject.toml
4 participants
@BenjaminLiuPenrose @thomasjpfan @rth @cmarmo
0