Opened 4 years ago

Closed 4 years ago

#21256 closed enhancement (fixed)

Upgrade and fix ipywidgets

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-7.4
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: c747d36 (Commits) Commit: c747d36b60294950fd4ccda3fc85662de288b2f9
Dependencies: #20218 Stopgaps:

Description (last modified by jdemeyer)

There are various issues with widgetsnbextension:

  1. It must be installed using pip. The typical python setup.py install does not work. See https://github.com/ipython/ipywidgets/issues/718. This is fixed by #20218.
  1. We need to enable the widgetsnbextension for ipywidgets to actually work.
  1. 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:

Change History (17)

comment:1 Changed 4 years ago by jdemeyer

  • Description modified (diff)

comment:2 Changed 4 years ago by jdemeyer

  • Branch set to u/jdemeyer/upgrade_ipywidgets_to_version_5_2_2

comment:3 Changed 4 years ago by jdemeyer

  • Commit set to 7aa94b615cc598fa1a708920ea0be4682e8c3f0f
  • Description modified (diff)
  • Summary changed from Upgrade ipywidgets to version 5.2.2 to Upgrade and fix ipywidgets

New commits:

7aa94b6Upgrade ipywidgets to version 5.2.2

comment:4 Changed 4 years ago by jdemeyer

  • Description modified (diff)

comment:5 Changed 4 years ago by jdemeyer

  • Dependencies set to #20218
  • Description modified (diff)

comment:6 Changed 4 years ago by git

  • Commit changed from 7aa94b615cc598fa1a708920ea0be4682e8c3f0f to b37edb2e44e3abd770a440704f5e384746f0e8da

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

b37edb2Upgrade and fix ipywidgets

comment:7 Changed 4 years ago by jdemeyer

  • Description modified (diff)

comment:8 Changed 4 years ago by git

  • Commit changed from b37edb2e44e3abd770a440704f5e384746f0e8da to 3fcd5b34fe0fa44c9a88fd8c66e39bc61ceb1dc0

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

6c9dffeUse pip to install most Python-based dependencies; see http://trac.sagemath.org/ticket/20218
63cade4This is no longer needed.
a692a30Update some packages that were added since the start of this branch to use pip.
3e16e00Some of the packages changed to use pip in 2fe1d357600e60d3422141ba5a7fb16efb3a691b needed pip added to their build deps.
332d915Move PIP_INSTALL into sage-env so that it's available so long as the Sage environment is activated
d6121b2pip install sagenb_exports
d7218a0Three new Python packages were added that should use pip for installation
3fcd5b3Upgrade and fix ipywidgets

comment:9 Changed 4 years ago by jdemeyer

  • Description modified (diff)

comment:10 Changed 4 years ago by git

  • Commit changed from 3fcd5b34fe0fa44c9a88fd8c66e39bc61ceb1dc0 to a7c97c7f5a9a2ea4652a6f3cc338cb55af595c0a

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

fd131c1Add --verbose flag to pip
afc90d4Add terminado as standard package
fdb403eAdd entrypoints and configparser packages
c4fc97cMerge remote-tracking branches 'trac/u/jdemeyer/add_a_terminado_package' and 'trac/u/jdemeyer/add_a_entrypoints_package' into t/20218/pip-install
7054c41Fix installation of Jupyter packages
a7c97c7Upgrade and fix ipywidgets

comment:11 Changed 4 years ago by git

  • Commit changed from a7c97c7f5a9a2ea4652a6f3cc338cb55af595c0a to a2213d9b04d223a934ca144fa5c37654c58fbfbd

Branch pushed to git repo; I updated commit sha1. New commits:

a2213d9Ignore warnings from ipywidgets

comment:12 Changed 4 years ago by git

  • Commit changed from a2213d9b04d223a934ca144fa5c37654c58fbfbd to 7a6c4c4209bdd172b83d46d68dfece65ed180d90

Branch pushed to git repo; I updated commit sha1. New commits:

7a6c4c4Remove deprecated widget abbreviations

comment:13 Changed 4 years ago by jdemeyer

  • Description modified (diff)
  • Status changed from new to needs_review

comment:14 Changed 4 years ago by git

  • Commit changed from 7a6c4c4209bdd172b83d46d68dfece65ed180d90 to 1fa20714d79a09ec2e9762507932deff9a965ab9

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

1fa2071Remove deprecated widget abbreviations

comment:15 Changed 4 years ago by git

  • Commit changed from 1fa20714d79a09ec2e9762507932deff9a965ab9 to c747d36b60294950fd4ccda3fc85662de288b2f9

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

99e1494Revert changes to numpy
f5c0b16wcwidth depends on pip
b137b56Upgrade and fix ipywidgets
38a0dd5Ignore warnings from ipywidgets
c747d36Remove deprecated widget abbreviations

comment:16 Changed 4 years ago by vbraun

  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

comment:17 Changed 4 years ago by vbraun

  • Branch changed from u/jdemeyer/upgrade_ipywidgets_to_version_5_2_2 to c747d36b60294950fd4ccda3fc85662de288b2f9
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.