-
Notifications
You must be signed in to change notification settings - Fork 28
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
Use the comm
package
#37
Comments
Actually not sure since |
cc @martinRenou who might know if this is related to a change in |
Yep bottom pins are tighter now, basically to get around the
structuredClone SNAFU. Need to bump the mock packages... Older 8.x and 7.x
ones will be compatible.
But also should be pinning for reproducible builds with front and backend.
As for comm.... Well that's not been very smooth either, and the ipykernel
dep is still carried by ipywidgets.
|
@martinRenou looks like the Which would help fix jupyter-widgets/ipywidgets#3819 |
Yes! |
Better do something, because jupyter-widgets/ipywidgets#3811 (comment) |
Problem
It should now be possible to use the new
comm
package: https://pypi.org/project/comm/Currently there is a mock for
ipykernel
here: https://github.com/jupyterlite/pyodide-kernel/tree/main/packages/pyodide-kernel/py/ipykernelWhich hopefully could be removed.
Proposed Solution
Use the
comm
package so can get rid of one of the mocks.Additional context
jupyterlite/jupyterlite#18
The text was updated successfully, but these errors were encountered: