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 debugger UI tests #11998

Closed
wants to merge 3 commits into from

Conversation

fcollonval
Copy link
Member

@fcollonval fcollonval commented Feb 3, 2022

References

UI tests on master are failing due to the debugger panel not opening when the notebook debugger is opened.

The bug is related to ipykernel 6.8.0 (I succeed to reproduce it after upgrading from ipykernel 6.7.0 to 6.8.0). Still looking for the root cause.

The initialized event does not appear on the kernel comm.

Code changes

User-facing changes

Backwards-incompatible changes

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@fcollonval fcollonval force-pushed the fix/debugger-ui-test branch from 188e83a to bcc559e Compare February 3, 2022 14:55
@fcollonval
Copy link
Member Author

@JohanMabille do you have any idea why the event initialized would not be emitted in ipykernel?

@fcollonval
Copy link
Member Author

Using ipykernel < 6.8.0 solves the issue. So definitely something to do with the latest ipykernel release.

@echarles
Copy link
Member

echarles commented Feb 4, 2022

These are the ipykernel PR merged between 6.7.0 and 6.8.0

ipython/ipykernel#816 and ipython/ipykernel#844 related to modules request (I have done them but don't see how it could impact)

ipython/ipykernel#849

ipython/ipykernel#850

@fcollonval
Copy link
Member Author

Commit b99e162 is testing the fix ipython/ipykernel#855

@fcollonval
Copy link
Member Author

No idea why b99e162 does not work - it works locally. So waiting for a patch release on ipykernel

@echarles
Copy link
Member

echarles commented Feb 4, 2022

It should work but I remember having weird behavior with explictit git revision number. You could try "git+https://github.com/ipython/ipykernel.git@master#egg=ipykernel"

@echarles
Copy link
Member

echarles commented Feb 4, 2022

@fcollonval
Copy link
Member Author

Fixed by upstream ipykernel v6.9.0 release

@fcollonval fcollonval closed this Feb 10, 2022
@fcollonval fcollonval deleted the fix/debugger-ui-test branch October 6, 2022 08:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2023
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