search

imbue-ai / jupyter_ascending

Ascend your Jupyter Notebook usage
MIT License
428 stars 26 forks source link

Add support for non-Python notebooks #25

Open dmalkr opened 3 years ago

dmalkr commented 3 years ago

jupyter_ascending don't support R-based notebooks, messaging ""Unable to find a paired notebook for test.sync.R in registered notebooks: {}".

Please add support for R-based notebooks.

zplizzi commented 3 years ago

Good point, this only supports python notebooks at the moment. Shouldn't be too hard to fix.

dmalkr commented 3 years ago

It seems it is need to write per-language analog of Communication plugin. Because it is need to send update queries to notebook kernel and then interpret it.

tfiers commented 2 years ago

julia crew represent

fecet commented 2 years ago

any progress?

statquant commented 2 years ago

I +10000 this feature request. We use R at work and we've actually been looking for a tool like this one for a long time! It looks incredible!

tfiers commented 2 years ago

@zplizzi, any pointers on how this should be implemented? Then someone can pick this up and contribute a patch

mbrucher commented 2 years ago

Strongly interested in knowing what needs to be done to have a R or a Q kernel running instead of Python. Looks like a few places where py is sent in stone, but would be easy to use a registration mechanism to not rely on something like ipykernel being present, encapsulate all calls to the comm so that we use the right one based on the extension and the builder would try to see if the extension is valid + check that the requested kernel is available?

zplizzi commented 1 year ago

We don't use non-python notebooks, so this isn't something I'll be adding - but if someone can figure out how to get this working without huge changes, I'm happy to review a PR.

fecet commented 1 year ago

We don't use non-python notebooks, so this isn't something I'll be adding - but if someone can figure out how to get this working without huge changes, I'm happy to review a PR.

In my setup, asceding only support default kernel so If open a notebook in a non-default conda environment it would not work. Am I doing wrong or miss something, otherwise this could be helpful for python guys

zplizzi commented 1 year ago

You'll need to install jupyter ascending in each kernel separately. Please open a separate issue if you have trouble with this!