Skip to content
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

Fix dependencies of ipywidgets #31523

Closed
mkoeppe opened this issue Mar 19, 2021 · 10 comments
Closed

Fix dependencies of ipywidgets #31523

mkoeppe opened this issue Mar 19, 2021 · 10 comments

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Mar 19, 2021

After #31278, which removed heavy dependencies from widgetsnbextension (which is a dependency of ipywidgets), we have to add some (lighter) dependencies to ipywidgets.

https://github.com/jupyter-widgets/ipywidgets/blob/master/setup.cfg#L35
lists:

install_requires =
  ipykernel>=4.5.1
  ipython>=6.1.0
  traitlets>=4.3.1
  nbformat>=4.2.0
  widgetsnbextension~=4.0a0
  jupyterlab_widgets~=2.0a0

These are not reflected in our dependency file. In this ticket we add the first three. (nbformat is only there so as to rule out some broken versions; we do not need jupyterlab_....)

So far this has not caused a build failure, but it is possible that one could appear in heavily parallelized builds.

The missing dependency is visible in the testsuite run for optional package p_group_cohomology in https://github.com/mkoeppe/sage/runs/2145415988

Running the test suite for p_group_cohomology-3.3.2...
Making check in src
Running doctests with ID 2021-03-19-18-58-47-d127e4fd.
Using --optional=debian,dochtml,meataxe,memlimit,pip,polylib,pycosat,sage,sage_spkg
Sorting sources by runtime so that slower doctests are run first....
Doctesting 13 files using 3 threads.
Traceback (most recent call last):
  File "/sage/src/bin/sage-runtests", line 182, in <module>
    err = DC.run()
  File "/sage/local/lib/python3.9/site-packages/sage/doctest/control.py", line 1212, in run
    self.run_doctests()
  File "/sage/local/lib/python3.9/site-packages/sage/doctest/control.py", line 909, in run_doctests
    self.dispatcher = DocTestDispatcher(self)
  File "/sage/local/lib/python3.9/site-packages/sage/doctest/forker.py", line 1658, in __init__
    init_sage(controller)
  File "/sage/local/lib/python3.9/site-packages/sage/doctest/forker.py", line 209, in init_sage
    controller.load_environment()
  File "/sage/local/lib/python3.9/site-packages/sage/doctest/control.py", line 530, in load_environment
    return import_module(self.options.environment)
...
  File "/sage/local/lib/python3.9/site-packages/sage/repl/ipython_kernel/all_jupyter.py", line 7, in <module>
    from .widgets_sagenb import (input_box, text_control, slider,
  File "/sage/local/lib/python3.9/site-packages/sage/repl/ipython_kernel/widgets_sagenb.py", line 31, in <module>
    from ipywidgets.widgets import (IntSlider, IntRangeSlider, FloatSlider,
  File "/sage/local/lib/python3.9/site-packages/ipywidgets/__init__.py", line 25, in <module>
    from .widgets import *
  File "/sage/local/lib/python3.9/site-packages/ipywidgets/widgets/__init__.py", line 4, in <module>
    from .widget import Widget, CallbackDispatcher, register, widget_serialization
  File "/sage/local/lib/python3.9/site-packages/ipywidgets/widgets/widget.py", line 18, in <module>
    from ipykernel.comm import Comm
ModuleNotFoundError: No module named 'ipykernel'
********************************************************************************
Error testing pGroupCohomology

CC: @dimpase @kliem @simon-king-jena @slel

Component: packages: standard

Author: Matthias Koeppe

Branch/Commit: 94fd11f

Reviewer: Jonathan Kliem

Issue created by migration from https://trac.sagemath.org/ticket/31523

@mkoeppe mkoeppe added this to the sage-9.3 milestone Mar 19, 2021
@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 19, 2021

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 19, 2021

Commit: 94fd11f

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 19, 2021

New commits:

94fd11fbuild/pkgs/ipywidgets/dependencies: Add missing dependencies

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 19, 2021

Author: Matthias Koeppe

@kliem
Copy link
Contributor

kliem commented Mar 23, 2021

Reviewer: Jonathan Kliem

@kliem
Copy link
Contributor

kliem commented Mar 23, 2021

comment:6

LGTM.

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 23, 2021

comment:7

Thank you.

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 29, 2021

comment:8

Setting priority to blocker to bring this ticket to the attention of the release bot.

@vbraun
Copy link
Member

vbraun commented Apr 2, 2021

Changed branch from u/mkoeppe/fix_dependencies_of_ipywidgets to 94fd11f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants