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

Give kernel connection options to a session connection #7820

Merged
merged 3 commits into from Jan 24, 2020

Conversation

jasongrout
Copy link
Contributor

References

This may help with jupyter-widgets/ipywidgets#2748

Code changes

Adds optional kernel connection options to a session connection, so (for now) you can say you want kernel connections that do not handle comms.

User-facing changes

None

Backwards-incompatible changes

This is backwards compatible because it introduces an optional argument.

@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

@jasongrout jasongrout changed the title Sessionkernelconnection Give kernel connection options to a session connection Jan 24, 2020
@jasongrout
Copy link
Contributor Author

@blink1073 - this may solve jupyter-widgets/ipywidgets#2748, so it may be good to get in to the RC if it passes review.

@blink1073 blink1073 added this to the 2.0 milestone Jan 24, 2020
Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@blink1073 blink1073 merged commit 0e0e7f2 into jupyterlab:master Jan 24, 2020
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 23, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pkg:help status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants