Workflow : Add nightly release of NumPy in linux workflows #13992
Workflow : Add nightly release of NumPy in linux workflows #13992rgommers merged 15 commits intoscipy:masterfrom
Conversation
|
Yup, good call. Turns out I missed two objects in |
|
Thanks @V0lantis, and thanks for the quick fix @BvB93. The numpy nightlies are built once a week, on Friday night typically, so a build which includes the fix will show up at https://pypi.anaconda.org/scipy-wheels-nightly/simple/numpy/ on Saturday. Easiest way to check is to use the version hash (everything after |
|
I have made the changes and tested upon the commit from @BvB93. it works. I pushed and next Saturday I will merge master to trigger a new workflows. |
|
Why is the mypy test being run before the actual test run itself? I'd like to see it the other way around so that we can see any test failures first, any typing failures are less consequential. In addition, 3.10 has now been released to beta. This CI run was originally written to identify issues with as-yet unreleased Python. Therefore, I think we need to amend the "Nightly" Action item to run off Python 3.10. If this is going to cause issues with the mypy run, then either we need to create a new Action, or move the mypy run to another Action item. |
Probably because mypy is >10x faster to run. Both need to pass, so it doesn't matter much.
+1 for upgrading to 3.10. It's orthogonal to this PR, but if GitHub Actions has 3.10 available it's a one-liner change. Can you try to include it here @V0lantis? If it doesn't work easily, let's do it in a separate PR. |
Yeap, going to try this |
Unfortunately, we cannot because numpy still doesn't have a release for Python3.10 : You can check the workflow here Maybe this should be added in the Nightly release @rgommers ? |
|
Ah yes, let's open a separate issue for upgrading the nightly to py3.10, and then work on first getting NumPy nightly wheels for 3.10 up. Not difficult, but it's an extra CI job there I think. |
|
The nightly CI used to build numpy from source, I don't think it took too long. |
|
Ah yes, going back to building from source should be okay as well. Indeed much faster than building scipy itself, so not a big issue. |
|
Ok so should I do an inplace build of numpy in the Nightly python job ? |
|
No, a regular |
This doesn't work, so I made a workaround. It is not pretty nor fast but it works fine. I tried with The failure with wget laid in the version of numpy with wget was : |
|
Okay I was being lazy, somehow the right invocation isn't easy to find. You need to use |
| sudo apt-get install -y --no-install-recommends python3.9-dev python3.9-distutils python3.9-venv | ||
| python3.9 -m pip install --upgrade pip setuptools | ||
| sudo apt install -y --no-install-recommends python3.10-dev python3.10-distutils python3.10-venv | ||
| curl -O https://bootstrap.pypa.io/get-pip.py && python3.10 get-pip.py && rm get-pip.py |
There was a problem hiding this comment.
So if this is necessary, it's a (temporary I guess) issue with the Python 3.10 that GitHub currently provides. Maybe add a comment that says that and points to this PR. We should be able to remove this later on.
|
Ok so for me, it ready to be merged |
Reference issue
This is a follow up to my PR #13876 which was failing once we merged it into master. I tested it on local and on my own fork and it seems to be working, at least I hope. You can found the workflow here
What does this implement/fix?
This PR takes the nightly release of Numpy to have the latest changes.
EDIT :
@BvB93 can you look to the
# type: ignore[attr-defined]statement forTestCaseimport (for example inscipy/optimize/_trustregion_constr/tests/test_qp_subproblem.py). I had the following errors :I wonder If it comes from numpy?