nteract / jupyter-paths

:city_sunrise: Pure JavaScript implementation of jupyter --paths --json
BSD 3-Clause "New" or "Revised" License
9 stars 11 forks source link

Use the implementation in Hydrogen #4

Closed n-riesco closed 8 years ago

n-riesco commented 9 years ago

I'm planning to add the flag --ijs-install=hydrogen to IJavascript so that it's possible to install the IJavascript kernel to one of the locations searched by Hydrogen.

To this end, I'm looking for a npm package that provides the functionality in jupyter-paths without requiring the installation of jupyter or ipython.

Hydrogen already does that here. I think it'd be great if that code in Hydrogen was moved into this repository.

rgbkrk commented 9 years ago

Heh, I wrote that code in Hydrogen but opted to make this one only Jupyter. I'm happy to add on only IPython paths but there's a lot of legacy kludge which I wasn't sure we had to bring over (mostly Windows related - "is this for XP or prior?")

n-riesco commented 9 years ago

For my needs, jupyter-only paths are fine; issue #2, though, is also requesting support for IPython.

Currently, IJavascript depends on IPython/Jupyter to install its kernel. Hydrogen can run kernels on machines where Jupyter hasn't been installed. I would like to be able to install the IJavascript kernel on those machines too.

rgbkrk commented 8 years ago

Hey @n-riesco, sorry I had not come back to this again. I put together ipython-paths to handle the old IPython paths.

Taking a step back though, are you asking for jupyter-paths to not rely on Python directly for pathing? I'd like to move jupyter-paths back to that and not shell out. What do you think?

rgbkrk commented 8 years ago

All set and done.