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

use <6 version of plotly for documentation and use latest for other purposes #136

Conversation

The-Broken-Keyboard
Copy link
Contributor

@The-Broken-Keyboard The-Broken-Keyboard commented Feb 12, 2025

Fixes #129 temporarily.

Changes code so that:

  • Doc builds with plotly version < 6
  • Other functionalities work with current plotly version

Issue is because of latest update of plotly, due to which the default interactive renderer ( plotly_mimetype+notebook ) of figure.show() in plotly is not supporting anymore in sphinx. While renderers like svg , png etc. are still supported.

When the issue gets fixed in plotly, sphinx or other related packages, then we can comeback to resolve it.


📚 Documentation preview 📚: https://arviz-plots--136.org.readthedocs.build/en/136/

@codecov-commenter
Copy link

codecov-commenter commented Feb 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.96%. Comparing base (15836bf) to head (789e79e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #136   +/-   ##
=======================================
  Coverage   78.96%   78.96%           
=======================================
  Files          28       28           
  Lines        3266     3266           
=======================================
  Hits         2579     2579           
  Misses        687      687           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aloctavodia aloctavodia merged commit 791d2f3 into arviz-devs:main Feb 12, 2025
3 checks passed
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 this pull request may close these issues.

Plotly plots not showing on the webpage for plotly >=6
3 participants