Skip to content

Plot gallery - #18

Open
sr-murthy wants to merge 55 commits into
mainfrom
plot-gallery
Open

Plot gallery#18
sr-murthy wants to merge 55 commits into
mainfrom
plot-gallery

Conversation

@sr-murthy

@sr-murthy sr-murthy commented Jun 19, 2026

Copy link
Copy Markdown
Member
  • Add kaleido to package deps + orca to docs deps
  • Add a plot gallery in the Sphinx docs - currently covers bar charts, bar line charts, descriptive tables, dual stack pyramid plots, frequency charts, Sankey diagrams, sunburst plots, upset plots (more will be added over time).

Visualisation page (RTD build version)

NOT TO BE MERGED until new releases are issued of Plotly and Sphinx Gallery - ref. this and this.

@sr-murthy sr-murthy self-assigned this Jun 19, 2026
@sr-murthy sr-murthy linked an issue Jun 19, 2026 that may be closed by this pull request
@sr-murthy
sr-murthy marked this pull request as draft June 19, 2026 14:32
@sr-murthy sr-murthy changed the title Plot gallery WIP: Plot gallery Jun 19, 2026
@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
isaricanalytics/analytics.py 11.33% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sr-murthy
sr-murthy force-pushed the plot-gallery branch 2 times, most recently from a82ad18 to 3518734 Compare June 19, 2026 14:42
@sr-murthy sr-murthy added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 1, 2026
@sr-murthy sr-murthy linked an issue Jul 1, 2026 that may be closed by this pull request
@sr-murthy
sr-murthy force-pushed the plot-gallery branch 6 times, most recently from 945432d to 83e85db Compare July 2, 2026 15:13
@sr-murthy sr-murthy added the packaging Packaging including management of project metadata and dependencies in the TOML. label Jul 2, 2026
@sr-murthy
sr-murthy force-pushed the plot-gallery branch 12 times, most recently from 4915521 to a09f389 Compare July 24, 2026 10:10
sr-murthy and others added 28 commits September 7, 2026 11:13
…s generated (matching thumbnails still not being generated)
Co-authored-by: Alasdair Wilson <60351846+alasdairwilson@users.noreply.github.com>
Signed-off-by: sr-murthy <sandeep.murthy@ndm.ox.ac.uk>
Co-authored-by: Alasdair Wilson <60351846+alasdairwilson@users.noreply.github.com>
Signed-off-by: sr-murthy <sandeep.murthy@ndm.ox.ac.uk>
Co-authored-by: Alasdair Wilson <60351846+alasdairwilson@users.noreply.github.com>
Signed-off-by: sr-murthy <sandeep.murthy@ndm.ox.ac.uk>
@sr-murthy

sr-murthy commented Sep 7, 2026

Copy link
Copy Markdown
Member Author

@EstebanGarciaG We can't merge this yet unfortunately because this branch uses another developer's Plotly fork to fix a Sphinx gallery issue I raised originally in relation to thumbnail generation in this PR (for the plot gallery in this branch). That issue was that Plotly (really, the Plotly scraper that Sphinx Gallery relies on) was generating placeholder thumbnails for the plots.

That developer proposed a solution/fix in this PR, which works. But the PR is being reviewed by the Plotly team, and needs to be approved.

So we need to wait for a new Plotly release. It is also likely that we need to wait for a new Sphinx gallery release, before we can merge this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation packaging Packaging including management of project metadata and dependencies in the TOML.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependencies do not support export of plots/figures docs: plot gallery

3 participants