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.