-
-
Notifications
You must be signed in to change notification settings - Fork 481
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 and fix ipywidgets #21256
Comments
This comment has been minimized.
This comment has been minimized.
Commit: |
This comment has been minimized.
This comment has been minimized.
New commits:
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Dependencies: #20218 |
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
Reviewer: Volker Braun |
Changed branch from u/jdemeyer/upgrade_ipywidgets_to_version_5_2_2 to |
There are various issues with
widgetsnbextension
:It must be installed using
pip
. The typicalpython setup.py install
does not work. Seepython setup.py install
command should error out and ask user to usepip
jupyter-widgets/ipywidgets#718. This is fixed by Use pip to install Python dependencies #20218.We need to enable the
widgetsnbextension
foripywidgets
to actually work.We add a patch to remove deprecated
@interact
abbreviations. This patch is applied upstream to version 6. To be more compatible with SageNB, we add this patch right now.Tarballs:
Depends on #20218
Component: packages: standard
Author: Jeroen Demeyer
Branch/Commit:
c747d36
Reviewer: Volker Braun
Issue created by migration from https://trac.sagemath.org/ticket/21256
The text was updated successfully, but these errors were encountered: