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

Update to JupyterLab 4.0.0b1 and 4.0.0b2 #6838

Merged
merged 16 commits into from
Apr 27, 2023
Merged

Conversation

@jtpio jtpio added this to the 7.0 milestone Apr 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2023

Binder 👈 Launch a Binder on branch jtpio/notebook/update-lab-beta-1

@jtpio
Copy link
Member Author

jtpio commented Apr 7, 2023

Strangely the title and checkpoint indicator seem to be gone:

image

@jtpio
Copy link
Member Author

jtpio commented Apr 7, 2023

Strangely the title and checkpoint indicator seem to be gone:

Looks like this piece of code is not hit on page reload:

const widget = new Widget({ node });
widget.id = DOMUtils.createDomID();
widget.addClass('jp-NotebookCheckpoint');
return widget;

Although the kernel logo using the same approach and shows up fine:

const widget = new Widget({ node });
widget.addClass('jp-NotebookKernelLogo');
return widget;

@jtpio
Copy link
Member Author

jtpio commented Apr 7, 2023

cc @fcollonval @brichet I see a few PRs related to the settings in the 4.0.0b1 changelog: https://github.com/jupyterlab/jupyterlab/releases/tag/v4.0.0b1

Maybe you know if they could be related to the issue mentioned above?

@brichet
Copy link
Contributor

brichet commented Apr 7, 2023

@jtpio yes it's strange.
I don't know if this is related but I can take a look at it next week.

@brichet
Copy link
Contributor

brichet commented Apr 11, 2023

@jtpio reverting the changes you mentioned at jupyterlab/jupyterlab#14195 (comment) seems that this does not fix it.

Another thing that is strange, both widgets (title and checkpoint) are displayed if the related plugin activation do not depends on IDocumentManager.
But in both case, the plugin is activated, depending or not in IDocumentManager.
Just in case this rings a bell...

EDIT:
if I replace optional: [IToolbarWidgetRegistry], by optional: [IToolbarWidgetRegistry, IDocumentManager], in kernelLogo, the widget is not displayed either.

@jtpio
Copy link
Member Author

jtpio commented Apr 11, 2023

Thanks @brichet for looking into it 👍

if I replace optional: [IToolbarWidgetRegistry], by optional: [IToolbarWidgetRegistry, IDocumentManager], in kernelLogo, the widget is not displayed either.

That's strange. At first glance I don't see why the IDocumentManager would have any effect here for toolbar items.

@brichet
Copy link
Contributor

brichet commented Apr 11, 2023

Actually, the problem comes from another part of jupyterlab/jupyterlab#14195:

https://github.com/jupyterlab/jupyterlab/blob/6d9217cc9681879dd822e6aaa5cedad2a648f99c/packages/settingregistry/src/settingregistry.ts#L394-L403

Everything works fine if I avoid this part for @jupyterlab/application-extension:top-bar plugin. And I don't know why it works if the IDocumentManager is not a dependency.

@jtpio
Copy link
Member Author

jtpio commented Apr 12, 2023

Maybe it's a race condition, or the behavior is undefined depending on which plugin is activated first?

@jtpio
Copy link
Member Author

jtpio commented Apr 12, 2023

Actually, the problem comes from another part of jupyterlab/jupyterlab#14195:

Maybe we should track this in a separate issue in JupyterLab so it can be fixed for the next JupyterLab beta release?

It looks like a breaking change which currently blocks the update of Notebook 7.

@jtpio
Copy link
Member Author

jtpio commented Apr 12, 2023

Maybe we should track this in a separate issue in JupyterLab so it can be fixed for the next JupyterLab beta release?

I opened jupyterlab/jupyterlab#14366 to track this.

@brichet
Copy link
Contributor

brichet commented Apr 13, 2023

There is a workaround for this update, waiting for it to be fixed in the next release of JupyterLab:
jupyterlab/jupyterlab#14366 (comment)

@jtpio
Copy link
Member Author

jtpio commented Apr 26, 2023

Beta 2 seems to be fixing the settings issue mentioned above 👍

Now the button to add a new cell at the end of the notebook needs to be fixed. For reference it was added in jupyterlab/jupyterlab#14109

image

@jtpio
Copy link
Member Author

jtpio commented Apr 26, 2023

Now the button to add a new cell at the end of the notebook needs to be fixed.

Might just try to disable it for now to be able to finish this update.

@jtpio jtpio changed the title Update to JupyterLab 4.0.0b1 Update to JupyterLab 4.0.0b1 and 4.0.0b2 Apr 26, 2023
@jtpio
Copy link
Member Author

jtpio commented Apr 26, 2023

bot please update playwright snapshots

@jtpio
Copy link
Member Author

jtpio commented Apr 26, 2023

bot please update playwright snapshots

@jtpio
Copy link
Member Author

jtpio commented Apr 26, 2023

Looks like we might hit jupyterlab/jupyterlab#14385 as well.

@jtpio jtpio marked this pull request as ready for review April 27, 2023 08:09
@jtpio
Copy link
Member Author

jtpio commented Apr 27, 2023

Opened #6852 to track adding back the "Click to add a cell" button. It was causing issues on CI with the reference snapshots on Firefox.

@jtpio jtpio merged commit 787339c into jupyter:main Apr 27, 2023
@jtpio jtpio deleted the update-lab-beta-1 branch April 27, 2023 08:12
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants