Does not work with the latest version of ipywidgets

[axil]

jupyter_bokeh does not work with ipywidgets >=8.0.0a0 Latest compatible version of ipywidgets is 7.7.2.

[axil]

Here's the relevant migration guide: https://github.com/jupyter-widgets/ipywidgets/blob/master/docs/source/migration_guides.md

Supposedly all that needs to be done is to update package.json accordingly.

[axil]

Tried to create a reproducible example in a Docker container. Couldn't (although it did appear in two machines, Linux and Windows). Closing as false alarm for the time being.

[axil]

Another bunch of tests revealed that the issue is not reproducible on Linux. It only affects Windows.

[axil]

I've found it out why it works on Linux: it is because it uses the global version of the notebook extension and not from the virtualenv. So it is basically the same on Windows and Linux.

Got it working with latest ipywidgets (8.0.2) by manually patching the index.js file (searched and replaced all instances of pWidget to luminoWidget and PhosphorWidget to LuminoWidget).

My suggestion is that updating package.json as described in the link above should suffice:

- "@jupyter-widgets/base": "^2 || ^3 || ^4",
+ "@jupyter-widgets/base": "^6",

Validating it I've run into the following problem:

> @bokeh/[email protected] build:lib /home/axil/upyter_bokeh
> tsc

src/plugin.ts:61:14 - error TS2322: Type 'Token<IJupyterWidgetRegistry>' is not assignable to type 'Token<any>'.
  Types have separate declarations of a private property '_tokenStructuralPropertyT'.

61   requires: [IJupyterWidgetRegistry],
                ~~~~~~~~~~~~~~~~~~~~~~

[axil]

@mattpap I've included the change proposed above into packages.json, built the project, tested against latest ipywigets 8.0.2 (works fine) and generated the package-lock and yarn.lock files.