jupyter / nb2kg

Other
73 stars 31 forks source link

tornado RuntimeError: finish() called twice #47

Closed nimengliusha closed 4 years ago

nimengliusha commented 4 years ago

When i try to connect remote kernel gateway through nb2kg, the kernel is not connected

Here is the notebook log: image

Any help or advice would be appreciated.

nimengliusha commented 4 years ago

Notebook v5.7.8, jupyter_kernel_gateway v2.4.1, nb2kg v0.7.0

KernelGateway log:

image

Ipynb view:

image

kevin-bates commented 4 years ago

Hmm - I haven't seen this, but also haven't used NB2KG in a while.

Are you in a position to try this on Notebook 6.x? Starting in 6.0, NB2KG is built into Notebook and can be "configured" simply by running:

jupyter notebook --gateway-url=${KG_URL}

You'll want to disable the NB2KG extension and remove the config settings that substitute the kernel manager, etc.

nimengliusha commented 4 years ago

i am using Notebook v5,7,8

kevin-bates commented 4 years ago

Yes - understood. I was hoping you could try with NB 6 as a datapoint.

kevin-bates commented 4 years ago

Hmm - I'm not able to reproduce this. I tried NB 5.7.8 with NB2KG 0.7.0 and hit both JKG (2.4.1) and JEG (master branch) servers, created python kernels, used them, and shutdown.

My launch command was as documented on the README (except I added --debug):

export KG_URL=http://127.0.0.1:8889
jupyter notebook \
  --NotebookApp.session_manager_class=nb2kg.managers.SessionManager \
  --NotebookApp.kernel_manager_class=nb2kg.managers.RemoteKernelManager \
  --NotebookApp.kernel_spec_manager_class=nb2kg.managers.RemoteKernelSpecManager --debug

Here are the versions used in the JKG case - all within the same env. nb578_nb2kg070_jkg241_requirements.txt

I suspect you might be using an older version of tornado than I am.

nimengliusha commented 4 years ago

Hi Kevin,

Thanks for your reply.

Upgraded the version of tornado from v6.0.1 to 6.0.4, still no luck. I'm going to try NB v6.0+

nimengliusha commented 4 years ago

Upgraded the version of NB from v5.7.8 to v6.0.3 is OK.

Thanks, Kevin