-
-
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
Kernel fails to send config sometimes #13164
Comments
Please do, so I can include it for 4.1.4.
Perhaps things could be solved if we moved to use the control channel instead of our own (dedicated) commsn channel? I say it because that channel is present by default (which could be part of the issue, since we have to create our comms channel). |
Thanks a lot @impact27! |
I think #13174 is a more reliable way of fixing this issue |
Ok, thanks for your work on this. I'll review it and try to include it for 4.1.4 |
Issue Report Checklist
conda update spyder
(orpip
, if not using Anaconda)jupyter qtconsole
(if console-related)spyder --reset
Problem Description
The kernel is supposed to send the config when opening:
https://github.com/spyder-ide/spyder-kernels/blob/05476b35cf9e55787521c43f91dd5eedcb56c7a2/spyder_kernels/comms/frontendcomm.py#L183
But sometimes it doesn't! And it is known because I added this line:
https://github.com/spyder-ide/spyder-kernels/blob/05476b35cf9e55787521c43f91dd5eedcb56c7a2/spyder_kernels/comms/frontendcomm.py#L163
Which solves this exact problem.
I found a workaround in #12134 that I could include in a standalone PR. But the problem remains: it should not be needed.
What steps reproduce the problem?
What is the expected output? What do you see instead?
This causes a comm timeout error. I have no idea why the kernel fails to send the config.
Solutions
1 - Figure out why the config is not sent
2 - Always send the config with every message
Versions
Dependencies
The text was updated successfully, but these errors were encountered: