Skip to content

doc: add thumbnails for skipped gallery and tutorial examples (fixes #17479)#31450

Open
ayeshakhalid192007-dev wants to merge 1 commit intomatplotlib:mainfrom
ayeshakhalid192007-dev:fix/add-missing-gallery-thumbnails
Open

doc: add thumbnails for skipped gallery and tutorial examples (fixes #17479)#31450
ayeshakhalid192007-dev wants to merge 1 commit intomatplotlib:mainfrom
ayeshakhalid192007-dev:fix/add-missing-gallery-thumbnails

Conversation

@ayeshakhalid192007-dev
Copy link
Copy Markdown

What Does This PR Do

Adds missing gallery thumbnails for examples that use the _sgskip suffix and are therefore excluded from documentation build execution. These examples were showing placeholder grey boxes in the gallery index page, which could confuse users into thinking the examples were broken.

Root Cause

Examples with _sgskip in their filename are intentionally skipped during the Sphinx Gallery build (because they require interactive input, external programs like FFmpeg, or intentionally take too long). Since sphinx-gallery never executes them, it cannot auto-generate a thumbnail image. Without a sphinx_gallery_thumbnail_path directive, these examples display an empty placeholder thumbnail.

Changes

  • Added # sphinx_gallery_thumbnail_path = '...' directive to 9 skipped examples
  • Created 6 new descriptive SVG thumbnails in doc/_static/, each designed to visually represent what the example demonstrates
  • Two examples reuse already-present images in _static/ (no new file needed):
    • image_thumbnail_sgskip uses stinkbug.png (an image being thumbnailed — fitting)
    • wire3d_animation_sgskip uses demo_mplot3d.png (existing 3D plot image)

Expected Impact

Gallery Section Before After
Animation / frame_grabbing Grey placeholder Film-strip animation thumbnail
Event Handling / ginput Grey placeholder Contour plot with cursor thumbnail
Event Handling / pong Grey placeholder Pong game court thumbnail
Misc / hyperlinks Grey placeholder Scatter with SVG link indicators
Misc / image_thumbnail Grey placeholder Real stinkbug image thumbnail
Misc / multiprocess Grey placeholder Producer/plotter process diagram
Misc / print_stdout Grey placeholder Terminal + line plot thumbnail
Mplot3d / wire3d_animation Grey placeholder 3D surface plot thumbnail
Widgets / lasso_selector Grey placeholder Scatter with lasso selection loop

Type of Change

  • Documentation / gallery improvement
  • Bug fix
  • New feature
  • Breaking change

Checklist

  • All new SVG thumbnails are original works with no licensing issues
  • SVG thumbnails visually represent what each example demonstrates
  • Follows the existing pattern set by multipage_pdf.py (sphinx_gallery_thumbnail_path)
  • No changes to example logic — only thumbnail directives added
  • Two examples reuse already-present _static/ images (no unnecessary new files)
  • Addresses the open items listed in Add thumbnails for tutorials/gallery where missing #17479

Fixes #17479

Several gallery examples that use the `_sgskip` suffix (excluded from
documentation build execution) were missing thumbnails, causing placeholder
grey boxes to appear in the gallery index page.

Add sphinx_gallery_thumbnail_path directives to 9 examples, along with
6 new SVG thumbnail images in doc/_static/ that visually represent
what each example does:

- animation/frame_grabbing_sgskip: film-strip animation thumbnail
- event_handling/ginput_manual_clabel_sgskip: contour plot with cursor
- event_handling/pong_sgskip: pong game court thumbnail
- misc/hyperlinks_sgskip: scatter plot with link indicators
- misc/image_thumbnail_sgskip: reuse existing stinkbug.png
- misc/multiprocess_sgskip: real-time producer/plotter diagram
- misc/print_stdout_sgskip: terminal + line plot thumbnail
- mplot3d/wire3d_animation_sgskip: reuse existing demo_mplot3d.png
- widgets/lasso_selector_demo_sgskip: scatter with lasso selection

Closes matplotlib#17479
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 4, 2026

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks. We also ask that you please finish addressing any review comments on this PR and wait for it to be merged (or closed) before opening a new one, as it can be a valuable learning experience to go through the review process.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide.
Please let us know if (and how) you use AI, it will help us give you better feedback on your PR.

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@rcomer
Copy link
Copy Markdown
Member

rcomer commented Apr 4, 2026

Hello @ayeshakhalid192007-dev. I see from your profile you have a strong interest in building autonomous AI agents. Was this PR AI generated?

@ayeshakhalid192007-dev
Copy link
Copy Markdown
Author

@rcomer Thanks for reviewing my PR. I appreciate your interest in my work on autonomous AI Agents. No, this PR was not AI-generated. I have been working on optimizing the current implementation, and these changes reflect some of those improvements.

Copy link
Copy Markdown
Member

@jklymak jklymak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me

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

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Add thumbnails for tutorials/gallery where missing

3 participants