-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
PR: Add command line option to connect to an existing kernel at startup (IPython console) #23444
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Social-Mean for your work on this!
Co-authored-by: Carlos Cordoba <[email protected]>
Hello @Social-Mean! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2025-01-15 17:34:01 UTC |
I need help! Spyder is not connected to the specific kernel. The reproduction process is as follows:
Now Spyder can start normally, but a new kernel is created as before, instead of connecting to the specified kernel. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Social-Mean, last comments then this should be ready.
And about what you said:
Start Spyder with --connect-to-kernel=kernel-*.json option
You don't need to use =
to pass command line options, so it needs to be
spyder --connect-to-kernel kernel-*.json
I tested that with my suggestion below and things are working as expected.
…get the absolute path to the file in the place where it's saved by Jupyter-client. Co-authored-by: Carlos Cordoba <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work on this @Social-Mean!
Note: I added an entry to our Changelog to mention the small API change you did here.
@meeseeksdev please backport to 6.x |
…to an existing kernel at startup (IPython console)
…ect to an existing kernel at startup (IPython console)) (#23471)
Description of Changes
Issue(s) Resolved
Fixes #23130
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct: Social-Mean