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

Jupytext & SOS kernel issue #453

Closed
thomaspernet opened this issue Mar 13, 2020 · 3 comments · Fixed by #455
Closed

Jupytext & SOS kernel issue #453

thomaspernet opened this issue Mar 13, 2020 · 3 comments · Fixed by #455

Comments

@thomaspernet
Copy link

Jupytext works fine with Python/R kernel but converts code cells into markdown cells when using the SOS kernel.

cf the image below. It is a code cell. After saving the notebook and restart it, it converted the code cell into markdown

Screenshot 2020-03-13 at 10 51 18

@mwouts
Copy link
Owner

mwouts commented Mar 13, 2020

Hello @thomaspernet , thanks for reporting this.
Maybe this is because the SOS kernel is not supported yet?

Anyway I'd be interested in adding it... Could you please provide a short Jupyter notebook with that kernel, and if possible, an example that reproduces the issue you encounter here?
Note that, if you like, you can contribute the example notebook in a PR, following these instructions.

Thanks

@thomaspernet
Copy link
Author

Dear @mwouts
thanks for your reply. Here is the link to download a reproducible notebook. I added the steps in the notebook with the link to install SOS-Kernel.

I hope it won't take too much of your time, or too complicated to add another Kernel..

Thanks!

@mwouts
Copy link
Owner

mwouts commented Mar 14, 2020

Thank you @thomaspernet for the notebook, that is very helpful!

I have added the SoS language in the add_sos branch, would you like to give it a try, and confirm that it works now? You can install the development version with

pip install git+https://github.com/mwouts/jupytext.git@add_sos

(dont forget to restart your notebook server before testing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants