-
Notifications
You must be signed in to change notification settings - Fork 65
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
Thebe integration: Modify sources #443
Comments
Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗 |
Thanks @HelgeGehring for the feedback! Agree completely. :) Working hard on getting there!! The immediate next step we are working on is to enable being able to import a figure from a notebook into an article: This is requiring quite a bit of rewiring of thebe in how it works with the theme, and how it keeps track of active kernels. I think that the editing is likely the next step in thebe support -- thoughts on that @stevejpurves? |
Sounds great! :) Thanks a lot! |
Yay, the first step seems to be done 🥳 Would you have a rough idea when the editing becomes a feature? This is the one thing holding me back from moving over from jupyterbook :) Thanks a lot for the great project! 😄 |
+1 this would be very useful! |
@HelgeGehring thanks for the question - we're refining how thebe integrates with the base myst-theme packages and components - where there is some flakiness and edge case failures still to work though. For example, around ipywidgets stability and how interactive outputs integrate into markdown documents -- e.g. for now that still requires power on + run, and trickily the current structure makes it difficult to do that with one click! So i think we'll need to do another pass on improving the code structure before adding more features like codemirrior + editing. I'd like to think that we can get to it in the next couple of months though! |
Sounds great, thanks a lot! For starting I'd already be happy to just edit the example and get new non-interactive output. One other minor thing I'm thinking of (I can make a separate issue):
Besides, thanks again for the great project! |
@rowanc1 @stevejpurves |
Hey @rowanc1 @stevejpurves ! Have you any update on this feature? I would really like to also use it in one of my project :) (Also congrats for mystmd!) |
The hesitance here is that the state management of notebook execution in So there are two options; wait for a refactor and introduce editing after we have better state management in place -- or -- drop editing in as-is and see how it behaves, perhaps as an opt-in beta feature and then fix as we go? |
Hmm, that sounds like an annoying problem @stevejpurves :/ any rough idea when we could hope for a refactor? Yeah, would be great to just give it a try and see what happens if we drop it in :) |
The effects are intermittent and race condition like, so definitely annoying. I hit them when trying to execute a notebook "quickly" after the session has started, so hit the power button and play close together, the effect is that the cells do not execute but the spinners run. There are similar effects occasionally when navigating that need page refreshes to fix. So the issues are not that huge, but you can see the hesitating to build on that foundation. If we try just dropping something simple in, I think there will be more feature requests pretty quickly :) to extend basic editing but maybe that helps push things along. Timing wise, 🤔 thinking... |
Which area is this feature request for?
Website User Interface
Describe the feature you'd like to request
Using jupyter-book it is possible to modify the source code after activating the thebe integration. Using mystjs, it seems that I can run the examples, but not modify them (which takes away the greatest advantage of thebe)
Describe the solution you'd like
Would there be a solution to change the source code fields to edit fields like in jupyter-book once "live code" is clicked?
Describe alternatives you've considered
No response
The text was updated successfully, but these errors were encountered: