-
Notifications
You must be signed in to change notification settings - Fork 37
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
ipypublish and jupytext #133
Comments
Thanks for the feedback @parmentelat. I'm glad you mention this, if you have a look at the contributors to MyST-NB, you may note that the mains ones are myself and @choldgraf (the maintainer of jupyter-book) 😁 The Executable Book Project is essentially a collaboration between ipypublish, jupyter-book and jupinx to take the best bits of our packages and combine them. This project is only a few months old, but we are looking to make a beta release soon, Note with the new package(s) we are specifically collaborating with jupytext and considering the use of
See spatialaudio/nbsphinx#420 for some outline/debate about the differences with nbsphinx, which has some overlapping but somewhat different goals. Also note at some point, once I'm happy that all the relevant functionality from ipypublish is available in this new package, I imagine I will probably look to "deprecate" ipypublish in some fashion. So yes it would be great if you have the time to check that out and give any feedback on any improvements you would desire to meet your use case 😄 |
hey @parmentelat - echoing what Chris mentioned - our goal is to (somewhat) reduce the forking-paths problem by unifying Check out beta.jupyterbook.org, and please do let us know if you have questions and such! |
@parmentelat Author of
It would be great if you could report your problems at https://github.com/spatialaudio/nbsphinx/issues! |
hiya everyone thanks for the feedback; I'll be looking into this new option, this looks promising; would anyone care to confirm what I think I gathered after this exchange
thanks again for any additional tip; I'll report here when I have more hands-on feedback with jupyter-book |
Quick response from me:
That's @chrisjsewell's call, but from my perspective, many of the ideas in ipypublish have now made their way into the new MyST-NB/MyST-parser/jupyter-book stack, and I think that stack will get more development attention moving forward.
Right now that is the case...some functionality was reduced because of the new build system but we could extend it to support some other use-cases, just want to do it intentionally!
Wanna open an issue about this in MyST-NB? To me it seems reasonable to support jupytext |
right; I'll do that, thank you |
I'm still happy to accept/merge PRs and look into "minor" bugfixes. But obviously there is only finite time in the day lol, so I don't intend to do anything more substantial with ipypublish.
Yeh its something we can discuss further on MyST-NB. In lieu of direct support is this not just a matter of calling |
Good point @chrisjsewell - as a stopgap, I added a section on converting files with jupytext here: https://beta.jupyterbook.org/content-types/jupytext.html |
well, one could make the same answer for executing the notebooks with nbconvert ;) |
@parmentelat ideally we'd like this to be possible with many different build engines, not just readthedocs. e.g., see https://github.com/ExecutableBookProject/cli/issues/44 and what will hopefully be a growing set of subsections here: https://beta.jupyterbook.org/start/publish.html |
Related to #133 (comment) -- we just converted stack of jupytext py files to myst, and the one gotcha is that the "language_info" metadata field is inferred when you're using py:percent, but if it's missing in a myst:md file sphinx raises a key_error. So I needed to go through and add the language_info to each of the converted myst notebooks. |
Perhaps you could make an issue for this, with the actual error raised. Because what you said is not exactly correct. When the notebook is run, the kernel adds the language_info to it. So the only reason it would not be present is if the file had not been executed. |
thanks for that hint: I had added |
I've just created #134; it's probably not something you'll accept as-is but I wanted to start the discussion as to how the config system could allow me to achieve this effect |
Hey @parmentelat thanks I will try and check it out over the weekend. |
I came over from nbsphinx, and the two things I missed at the start were parsing |
I'm preparing for the courses that start in september; I have several courses that were formerly ipynb and I want to migrate to some text format because the json format is a pain wrt git, both for teachers and for propagating course changes to students with the previous workflow, some of the contents were ipypublish-ready, and ended up on readthedocs
so for mitigating the risk I am seeing ipypublish as a fallback; which if you will allows me to split the move into 2 smaller steps: moving to myst/ipypublish, and moving to jupyter-book later on; at this point in time I'll soon have to accept that I won't be able to swallow both steps in time.. you've made your position very clear regarding ipypublish, that's why I made the PR as small as possible, please let me know how you feel about it |
Oh yeh that’s, I should be able to look at the PR over the weekend. It’s not to say you should be using Jupyter-book over ipypublish, just interested to know what your perspective was |
Is your feature request related to a problem? Please describe.
my use case is pretty simple
I write notebooks for teaching, mostly rise-powered slideshows
I also use ipypublish to produce handouts on readthedocs
recently I have decided to go for jupytext, it is so much simpler for me to store my contents under git as either .py or .md files
however I have not been able to keep on using ipypublish after moving to jupytext
truth be said :
Describe the solution you'd like
first off I'd like to know if that is something that is supposed to be easy to get to work, or not
Describe alternatives you've considered
I have given a shot at
nbsphinx
, with which I was more successful wrt jupytext (I guess I found the recipe for binding .py extension) but am running into problems of other kinds (images for example seem to be a struggle; also I had created a custom thingy for having the slide notes showup as a custom class); it's be much easier for me to be able to stick to one solution for this need; it's rather crucial, but otoh the whole point would be ideally to not have to worry about this sort of things to much ;-)Additional context
I have not kept track of my early attempts, that I scratched when going to nbspinx if relevant I can put that back in shape and provide more details
as an aside, and more generally, I have to say that I am quite confused by the wide spectrum of possibilities for doing this sort of things (ipypublish, nbsphinx, jupyter-book ?, this new myst-nb thing I've [come across recently from here]), that are intimidating b/c of number of options, the pace of change, etc.. if anyone has the time and knowledge to write a short overview on all these within a bit of context, it would honestly be tremendously useful IMHO
thanks in any case for the great tool :)
The text was updated successfully, but these errors were encountered: