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

Watch kernel and runtime directories for updates #35

Open
rgbkrk opened this issue May 25, 2015 · 6 comments
Open

Watch kernel and runtime directories for updates #35

rgbkrk opened this issue May 25, 2015 · 6 comments
Labels
enhancement 🌟 New feature ideas good first issue Good for new contributors

Comments

@rgbkrk
Copy link
Member

rgbkrk commented May 25, 2015

We could use fs.watch to track kernel installations (or other runtimes being launched, e.g. the notebook). To make this simpler, there are higher level watch libraries like https://github.com/mikeal/watch or https://github.com/bevry/watchr.

This would make it so it isn't just determined once on startup, but asynchronously throughout use.

@willwhitney willwhitney modified the milestone: v0.5.0 May 26, 2015
@willwhitney willwhitney removed this from the next release milestone Dec 14, 2015
@BenRussert
Copy link
Member

@rgbkrk I wonder if (almost two years later) you have any more thoughts on this? Is there anything in the nteract repo that might be a good model?

@rgbkrk
Copy link
Member Author

rgbkrk commented Mar 23, 2017

Sidecar detects new runtime directories, though the code has grown stale (no one hacking on it, lots of outdated libraries, no shared knowledge of what's going on in there). For Hydrogen, I'd write the kernel checking all over again or make it part of kernelspecs.

@lgeiger
Copy link
Member

lgeiger commented Mar 23, 2017

👍 For using kernelspecs!

Though at the moment it doesn't detect the default python installation so users need to run python -m ipykernel install --user so it can be detected.

I'm wondering if the easiest thing would be, if kernelspecs ran python -m ipykernel install --user if it can't detect a python kernel and then rerun current detection method.

@rgbkrk
Copy link
Member Author

rgbkrk commented Mar 23, 2017

We could bring the default python detection to kernelspecs though, which is currently buried away in old PRs of mine on nteract/nteract that I've closed.

@lgeiger
Copy link
Member

lgeiger commented Mar 23, 2017

We could bring the default python detection to kernelspecs though, which is currently buried away in old PRs of mine on nteract/nteract that I've closed.

Ahh sure I forgot!

For reference: nteract/nteract#1384

@lgeiger
Copy link
Member

lgeiger commented Oct 15, 2017

Atom now has a file watching API. Since we're now using kernelspecs we could easily watch the directories returned by jupyter-paths for changes and update the kernels.

I'd be very happy if someone is interested in implementing this.

@lgeiger lgeiger added enhancement 🌟 New feature ideas good first issue Good for new contributors labels Oct 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🌟 New feature ideas good first issue Good for new contributors
Projects
None yet
Development

No branches or pull requests

4 participants