-
Notifications
You must be signed in to change notification settings - Fork 132
docs: adding plottly support to gallery examples #2346
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
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2346 +/- ##
=======================================
Coverage 88.29% 88.29%
=======================================
Files 187 187
Lines 14819 14819
=======================================
Hits 13084 13084
Misses 1735 1735 🚀 New features to boost your workflow:
|
Currently it is needing something on ansys-sphinx-theme. Nothing we can do about this at the moment. Converting in draft. |
If depends on |
Closing for the moment. |
…tly-support-to-docs
Still it does not work... I get a white space instead of the plot. I did build the documentation using the CICD, download the html artifact and open it with the browser. I believe, I do not require to have a live http server.... But I might be wrong |
Could you try this config, @germa89? https://github.com/plotly/plotly-sphinx-gallery/blob/9a82c6de451843cd86368a07770adc661f2918fe/docs/conf.py#L187-L198
|
Testing this in dbb3516
|
@jorgepiloto it is still broken... Maybe I am doing something wrong? Can you check the example and/or configuration? |
Closing because of stale. @jorgepiloto |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is the plot I obtain when downloading the PR's artifacts (973c746): As visible on the following screenshot, the widget are working even though the initial plot is a little bit cropped: |
@Revathyvenugopal162 now you know the struggle! 🤣 |
Safari shows nothing on the artifacts. Edge and Chrome (same engine) shows the graph cropped. I am going to merge and see how the deployed docs looks like (which is what matters). If it does not look good, I will revert the merge. |
As the title.
Reference: https://sphinx-gallery.github.io/stable/auto_plotly_examples/plot_0_plotly.html#sphx-glr-auto-plotly-examples-plot-0-plotly-py
Close #1211