-
-
Notifications
You must be signed in to change notification settings - Fork 523
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
Upgrade NumPy to 1.19.1, scipy to 1.5.2, networkx to 2.4, add pybind11 package #29766
Comments
This comment has been minimized.
This comment has been minimized.
Branch: u/dimpase/packages/networkx24 |
Last 10 new commits:
|
Dependencies: #29425 |
Commit: |
Author: Dima Pasechnik |
comment:2
You may get more stuff if you are using scipy 1.4 with that. I patched all the deprecation I could in sage-on-gentoo |
comment:3
for scipy 1.4 (cf #29425 - which is a dependency of this ticket!) I chose to |
comment:4
Minor (perhaps trivial) point, but pulling it from that link gave a zip file named With this ticket, I get two deprecation warnings in
|
comment:5
Could you post deprecation warnings you get? Cause I thought these were fixed on #29425 |
comment:6
Both of them are like this:
|
comment:7
When I run the file as a standalone, I get
|
comment:8
You shouldn't have this warning with networkx 2.4. It implies you are still using 2.2. |
comment:10
@kiwifb I am using 2.4 as far as Sage knows:
@dimpase I didn't realize the entire branch of #29425 had not been merged in. It looked like it had from a quick glance. Once I did that then indeed I do not get the deprecation warnings. |
comment:11
Since the doctests do pass for me with #29425 (and you don't need to merge it in any further I think), positive review. |
Reviewer: Travis Scrimshaw |
comment:12
Replying to @tscrim:
I don't quite understand what "pulling it from that link" means. The idea of And this automation works for me on Linux. The ULR is copied verbatim from https://github.com/networkx/networkx/releases/tag/networkx-2.4 |
comment:13
Replying to @dimpase:
Okay, I tried with just |
This comment has been minimized.
This comment has been minimized.
comment:15
From #29425: running the testsuite of networkx requires pytest. |
comment:115
Replying to @dimpase:
It keeps "nose pytest" only when running the Because these are testsuite-only dependencies. |
comment:116
Replying to @mkoeppe:
but somehow it does not work in the GH Actions realm (or maybe at all), no? |
comment:117
Yes, there is a problem - thanks for catching it. I am running GH Actions build with Let's change that. |
comment:118
Replying to @dimpase:
Sure, let's do it. |
This comment has been minimized.
This comment has been minimized.
comment:121
Tests run at https://github.com/mkoeppe/sage/actions/runs/190765555 |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:125
Replying to @mkoeppe:
That was not quite the right explanation, but now I have fixed it. Subtleties of Makefile variables and who overrides whom. |
comment:126
Let's please get this in? |
comment:127
lgtm |
comment:128
Thank you! |
Changed branch from u/mkoeppe/packages/networkx24 to |
comment:130
Why is there no SPKG.rst file for pybind11? Are these suddenly optional? |
Changed commit from |
comment:131
Replying to @jhpalmieri:
A simple mistake. There's actually nothing in our scripts that checks the presence of these files (see e.g. #29334). Let's fix it in #30357. |
comment:132
Somehow this broke building documentation on macOS. See #30351. |
comment:133
I have no idea why this would broke docbuilding, but that's what it looks like. Does anyone have ideas how to debug it? Please discuss at #30351. |
Follow-up from the update of numpy to 1.16.6 (#29429):
We update
All of the above versions support Python >= 3.6.
One deprecation warning is removed here, another suppressed (from #29425),
the rest it trivial.
Tarballs: see checksums.ini [upstream_url]
Depends on #29929
Upstream: Fixed upstream, but not in a stable release.
CC: @antonio-rojas @mkoeppe @kiwifb @isuruf @slel @dcoudert @tscrim @fchapoton @alexjbest
Component: packages: standard
Author: Dima Pasechnik, Matthias Koeppe
Branch:
5d4b013
Reviewer: Travis Scrimshaw, Isuru Fernando, Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/29766
The text was updated successfully, but these errors were encountered: