-
Notifications
You must be signed in to change notification settings - Fork 7
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
Pyodide seems to interfere with output of r plotly plots #47
Comments
Possibly duplicate of #44 if the underlying mechanism that prevents altair and plotly plots from showing is the same. Do plotly plots in python with pyodide out fine? |
As of commit cbacfc4 this should work better. I've replaced using the requirejs library with loading the Jupyter This works OK for now, but will need to be revisted when we start to implment more of the jupyter widget functionality, since several widgets assume requirejs is available for use. |
@georgestagg I think this is probably related to some of AMD loading hiccups that was plaguing monaco; Can you boost the default version of Pyodide up to 0.26.2? |
The issue we're seeing is not due to Pyodide per se, it just seems that way because we were only including the requirejs library when If I remove the gate, so that requirejs is always loaded into the page, even standard non-WebAssembly R HTML widget features are broken. Here Pyodide isn't even loaded into the page at all, but requirejs is. So, the problems are wider than something in Pyodide. I think I remember reading some time ago that requirejs just simply does not work well with Quarto -- but I can't find that thread now so it might be old or mis-remembered news. |
Well it's true that Quarto is very heavy on JS & CSS dependencies. Though, I'm surprised that Quarto completely obliterates it. Probably time to briefly look through the partial template to see the built-in dependencies. |
Here is one quarto issue that mentions problems with require.js in the context of using a jupyter kernel, which I noticed goes away when using a knitr kernel quarto-dev/quarto-cli#3424. Not sure if it is related but thought I would mention it just in case |
Thanks, I believe that was the thread I was thinking of. My knowledge was misremembered, though. Carlos’s comment suggests that it’s OJS that requirejs won’t work well with, not Quarto itself, since it replaces the For now, and probably for the near-to-mid future, I think we can just get away with not including requirejs. I don’t think we’ll need it until we’re wanting to support advanced custom Jupyter Widgets, and we don’t even have the standard widgets communicating back with the Python session yet. |
Problem
Inclusion of pyodide code chunks appears to interfere with plotly output in quarto doc. Not sure if this should be posted here or on the Quarto page.
Expected behaviour
Plotly plots output as normal
Actual behaviour
Plotly plots not output in rendered doc when pyodide code chunk is included as below. Plotly plots also not output in rendered doc when pyodide code chunk is included and eval=FALSE. Plotly plots output as expected when pyodide code chunk removed entirely.
The text was updated successfully, but these errors were encountered: