-
Notifications
You must be signed in to change notification settings - Fork 948
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
Installing the ipywidgets for jupyterlab breaks #2006
Comments
Do you happen to know what the incompatibility was? |
Some issue with the model. Not sure on what specifically it was. https://github.com/jupyter-widgets/ipywidgets/blob/master/packages/base/src/manager-base.ts#L347 "Could not instantiate widget" is the error I get. Note that if I update ipywidgets to 7.1.2 - it gets fixed. The combination that worked a few months ago - ipywidgets==7.0.0 and jupyter-widgets/[email protected] doesn't seem to work now |
Was there another error shown? It looks like it prints out that, and then throws the real error. |
I can't even get jupyterlab 0.27 running - apparently it depends on the latest jupyterlab_launcher, which is maybe incompatible? |
Hm, I'm not sure on this. We have a fairly old installation of jupyterlab from last September @rsinha25 - any comments? |
(apparently it depends on jupyterlab_launcher 0.4.0 - explicitly installing that makes jlab 0.27.0 work). |
I just duplicated the error. I see:
This comes because the controls package was upgraded to be protocol version 1.1.0, but the python side still is looking for just protocol version 1.0.0 (see
Here are a couple of possible ways to deal with this (in a semi-technical ramble of thoughts):
|
Ah. I do see the semver error too. I thought that was an error coming up because of a previous error and disregarded it. |
Yes, I think it's the dependency on controls that is the solution. |
Let's see if we can at least put in place a solution going forward (especially since we are going to be bumping the controls version again for 7.2). |
In the current jupyterlab-manager, we take care of this issue by registering controls under both the current version and previous versions, but it's a manual process and doesn't work so well with old jupyterlab-managers. Releasing 7.2 will run into the same issue - the current jupyterlab-manager will register control spec 1.0.0 and 1.2.0, but not 1.1.0. Perhaps we should release a patch release of jupyterlab-manager to add 1.1.0 registration, again as a stop-gap. @SylvainCorlay - let's rehash this version stuff (yet again)... |
Actually, let's continue the discussion on #2017, where I've made a change. |
…ls package. This resolves the backwards-incompatibility issues for other libraries with applying this logic to all modules, but still fixes jupyter-widgets#2006.
Wanted to understand if:
This is happening For the 1.1.0 fix too? |
The version we are releasing soon (like in the next few days, hopefully, compatible with jlab 0.31, i.e., jlab beta 1) should handle 1.0.0, 1.1.0, and 1.2.0. Is that what you are wondering? |
(In other words, the jlab manager we are releasing in the next few days should work with ipywidgets 7.0, 7.1 and 7.2.) |
Yes. I think that would resolve it =) Would I be forced to upgrade jlab from 0.27.0 to 0.31.0 ? |
The new jlab-manager is only compatible with jupyterlab 0.31.x. Is it crucial to you to stay on jlab 0.27? That is quite old at this point, and both that version of jlab and that version of jlab-manager were considered alpha. |
XD yeah it's required for me to continue using the alpha version for now. |
Thanks - that's probably the easiest solution for everyone involved at this point. |
We are using
ipywidgets
7.0.0 in our systems and are usingjupyter-widgets/jupyterlab-manager
v0.27.0 withjupyterlab
0.27.0It looks like there was a
jupyter-widgets/base
v1.1.8 which is incompatible with these but the previousjupyer-widgets/jupyterlab-manager
is using ^1.0.1 implying any system which is trying to usejupyter-widgets/jupyterlab-manager
will now build withjupyter-widgets/base
v1.1.8 and will be broken.Seems to us like the
jupyter-widgets/base
which seems to be backward incompatible has been released as a 1.x series and braking other packages - as this is backward incompatible, it should have been a 2.x series ?The text was updated successfully, but these errors were encountered: