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

Fix qtconsole issues #638

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

davidbrochart
Copy link
Member

@davidbrochart davidbrochart commented Apr 8, 2021

@ccordoba12 this should fix the issues you were having with v6.1.13 in spyder. I checked that qtconsole works fine, but it might be worth trying spyder too. Actually I should open PRs in upstream projects with this jupyter_client.

@davidbrochart
Copy link
Member Author

I've been opening PRs in Voila, nbconvert and qtconsole, and this seems to be going well. I'm going to merge this PR and make a release, if there is no objection.

@ccordoba12
Copy link
Contributor

Hey @davidbrochart, thanks a lot for your prompt help! Could you give me a couple of hours before merging? I'd like to run Spyder's test suite against this too, just to be sure things are ok on our side.

@davidbrochart
Copy link
Member Author

Sure, take your time, it's night time over here so not planning to do it until tomorrow.

@davidbrochart
Copy link
Member Author

@ccordoba12 I tried spyder and fixed another issue. Could you please confirm everything works fine on your side?

@ccordoba12
Copy link
Contributor

ccordoba12 commented Apr 9, 2021

Sorry for the delay @davidbrochart. I'm running our test suite against this branch here: spyder-ide/spyder#15205.

@ccordoba12
Copy link
Contributor

I'm getting this error in some of our console tests:

ERROR    asyncio:base_events.py:1619 Exception in callback BaseAsyncIOLoop._handle_events(413, 1)
handle: <Handle BaseAsyncIOLoop._handle_events(413, 1)>
Traceback (most recent call last):
  File "/usr/share/miniconda/envs/test/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
RuntimeError: cannot enter context: <Context object at 0x7f099c46df00> is already entered
ERROR    asyncio:base_events.py:1619 Exception in callback BaseAsyncIOLoop._handle_events(411, 1)
handle: <Handle BaseAsyncIOLoop._handle_events(411, 1)>
Traceback (most recent call last):
  File "/usr/share/miniconda/envs/test/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
RuntimeError: cannot enter context: <Context object at 0x7f099c4ec050> is already entered
ERROR    asyncio:base_events.py:1619 Exception in callback BaseAsyncIOLoop._handle_events(413, 1)
handle: <Handle BaseAsyncIOLoop._handle_events(413, 1)>
Traceback (most recent call last):
  File "/usr/share/miniconda/envs/test/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
RuntimeError: cannot enter context: <Context object at 0x7f099c46df00> is already entered

Reference: https://github.com/spyder-ide/spyder/pull/15205/checks?check_run_id=2307155447

@davidbrochart
Copy link
Member Author

Thanks for reporting @ccordoba12, it seems to be fixed now: spyder-ide/spyder#15217

@ccordoba12
Copy link
Contributor

ccordoba12 commented Apr 9, 2021

Yep, that seems to be the case. Then I'd say we're ready to merge this one.

Thanks again for your quick response to fix the situation and help us to be compatible with the future jupyter_client. We really appreciate it.

@davidbrochart davidbrochart merged commit c5aceba into jupyter:master Apr 12, 2021
@davidbrochart davidbrochart deleted the fix_threaded_client branch April 12, 2021 07:04
@stonebig
Copy link

stonebig commented Apr 13, 2021

is it normal that jupyter-client-6.2.0 is no more published on pypi ?
image

@SylvainCorlay
Copy link
Member

SylvainCorlay commented Apr 13, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants