-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Render is failling with latest Pweave #65
Comments
Maybe this is related: I think that WorkerManager is doing something with the workers that cause it the thread to change, or maybe is because it's use QThread |
I've finally found the bug, It's caused because the workermanager change the worker thread in _start() method https://github.com/spyder-ide/spyder/blob/master/spyder/utils/workers.py#L264 Using instead the logic for ProcessWorker, fix the bug (although that doesn't seem as the best solution) cc: @goanpeca |
I think that the only solution is to move to use |
@rlaverde moving forward with spyder handling envs, and other python versions, we will most likely rely on the LSP or the process worker since, the python worker only really makes sense for things that will be executed in the same environment that spyder lives. I know this is a different issue, but it is probably more reliable to do it using the process worker, although it might take a little more time (while a new python process is started). You can also have a helper script that runs PWeave and execute that script in a process if that makes things easier. |
Render is failing with:
signal only works in main thread
after this pweave change: mpastell/Pweave@3765b0cThe text was updated successfully, but these errors were encountered: