-
-
Notifications
You must be signed in to change notification settings - Fork 490
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
IPython/Jupyter upgrade for Sage 9.5 #31280
Comments
Branch: u/mkoeppe/jupyter_upgrade |
Commit: |
This comment has been minimized.
This comment has been minimized.
New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:4
|
comment:5
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
Author: Matthias Koeppe |
comment:9
With this update I seem to be running into:
|
comment:10
I don't see why this would have anything to do with Jupyter. The problem seems to happen during the build of the pplpy documentation with sphinx. Probably to make cross linking, sphinx requires |
comment:11
This ticket is updating 40 packages, including |
Work Issues: rebase on #31328 |
Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
Changed reviewer from Dima Pasechnik, https://github.com/mkoeppe/sage/actions/runs/1176095684, https://github.com/mkoeppe/sage/actions/runs/1177591365, ... to Dima Pasechnik |
comment:88
Replying to @mkoeppe:
Now |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Changed reviewer from Dima Pasechnik to Dima Pasechnik, https://github.com/sagemath/sagetrac-mirror/actions/runs/1207522574 |
Changed reviewer from Dima Pasechnik, https://github.com/sagemath/sagetrac-mirror/actions/runs/1207522574 to Dima Pasechnik |
comment:91
OK, this looks good. I see you're testing cygiwn on GH Actions. Feel free to flip to positive review if these make sense to you. |
comment:92
Thank you! Yes, the Cygwin test looks fine (in previous runs, the multi-stage CI for Cygwin had revealed some of the missing dependencies). |
comment:93
Should be a dependency of ipykernel as far as I can tell. |
comment:95
It is only needed at runtime. Being a standard package, Since this is already on Volker's branch, let's keep it on "positive review" and add the dependency in a follow-up ticket |
comment:96
I think I got the error after I typed |
comment:97
Let's do this in #32493 |
Changed branch from u/mkoeppe/jupyter_upgrade to |
Changed keywords from none to upgrade, jupyter |
This comment has been minimized.
This comment has been minimized.
Changed commit from |
|
https://repology.org/projects/p/?inrepo=sagemath
We upgrade the following packages
(updating some SPKG.rst as needed):
Previous updates: #26919, #31373, #31476, #31648.
PR to avoid pulling in a backport package: jupyter/nbclient#154 (merged)
Depends on #32371
Depends on #32372
CC: @jcamp0x2a @slel @kcrisman @egourgoulhon @kiwifb @videlec @kliem @antonio-rojas @isuruf
Component: packages: standard
Keywords: upgrade, jupyter
Author: Matthias Koeppe
Branch:
e2fb5ce
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/31280
The text was updated successfully, but these errors were encountered: