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 extremely long kernel restart times #119

Closed
jbweston opened this issue Dec 19, 2018 · 8 comments
Closed

Fix extremely long kernel restart times #119

jbweston opened this issue Dec 19, 2018 · 8 comments
Labels

Comments

@jbweston
Copy link
Contributor

(original issue on GitLab)

opened by Bas Nijholt (@basnijholt) at 2017-09-07T13:20:38.839Z

Not sure why this happens, but kernel restarts can take >30 seconds sometimes.

Probably running the notebook local and looking at the terminal will tell you more.

@jbweston
Copy link
Contributor Author

originally posted by Bas Nijholt (@basnijholt) at 2017-09-15T13:07:49.356Z on GitLab

Potentially fixed by cd089d9 and 2d931fe.

@jbweston
Copy link
Contributor Author

originally posted by Joseph Weston (@jbweston) at 2017-10-27T13:02:19.396Z on GitLab

The problem is indeed to do with the fact that asyncio notebook integration does not play nicely with concurrent.futures.Executor.

I have opened an upstream issue here,

but the maintainers do not seem very active

@jbweston
Copy link
Contributor Author

originally posted by Bas Nijholt (@basnijholt) at 2018-01-02T13:04:29.324Z on GitLab

There is an issue that addresses this problem here ipykernel/issues/291.

@jbweston
Copy link
Contributor Author

originally posted by Joseph Weston (@jbweston) at 2018-02-14T09:09:32.566Z on GitLab

@basnijholt did we check that the master version of ipykernel fixes everything?

@jbweston
Copy link
Contributor Author

originally posted by Bas Nijholt (@basnijholt) at 2018-02-14T09:55:33.437Z on GitLab

The fix is even in the latest released version :)

@jbweston
Copy link
Contributor Author

originally posted by Joseph Weston (@jbweston) at 2018-02-19T14:33:51.512Z on GitLab

Seems fixed with the versions of ipykernel and jupyter_client that we require

@jbweston
Copy link
Contributor Author

originally posted by Bas Nijholt (@basnijholt) at 2018-02-19T14:36:04.195Z on GitLab

Actually, there is still this issue that is open.

The notebook processes get cleaned up correctly, but the restarting is still slow.

I'll reopen and add a 'Blocked' label.

@jbweston
Copy link
Contributor Author

originally posted by Bas Nijholt (@basnijholt) at 2018-12-07T19:13:24.210Z on GitLab

This seems to be fixed for quite a while now.

jhoofwijk pushed a commit to jhoofwijk/adaptive that referenced this issue Apr 17, 2019
Resolve "(Learner1D) add possibility to use the direct neighbors in the loss"

Closes python-adaptive#119

See merge request qt/adaptive!131
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant