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 discourse chat 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.

Could you take a pass through the examples and see if others are missing? For example, https://matplotlib.org/stable/gallery/mplot3d/rotate_axes3d_sgskip.html also does not have a thumbnail

Use animation (FuncAnimation) which doesn't produce a static frame

Can you explain this b/c most of our animation examples produce a thumbnail? Also, can you explain how you think your images illustrate the example they're for?

Thanks everyone for the reviews! I've cleaned up the PR quite a bit based on the feedback:

@timhoffm — You're right about the PR template and AI disclosure. I've updated the PR description now to follow the template properly and added an AI disclosure statement. Thanks for noting that.

@scottshambaugh — I added the rotate_axes3d_sgskip thumbnail as you suggested. Good catch on that one.

@story645 — Great questions:

1. "Most of our animation examples produce a thumbnail" — You're right that animation examples using FuncAnimation typically do produce thumbnails because sphinx-gallery captures the last rendered frame. The ones here are different:
   • frame_grabbing_sgskip uses MovieWriter directly (not FuncAnimation), bypassing the normal rendering pipeline
   • wire3d_animation uses a manual loop with time.sleep() instead of FuncAnimation, so there's no automatic frame capture
   • Both are _sgskip examples, so they're never executed during doc builds anyway
2. "How do the images illustrate the example they're for?" — The thumbnails show a representative output frame from each example. For instance, the frame_grabbing.png shows the sine wave plot that the example generates, pong.png shows the actual pong game interface, etc.

I've also rebased the branch to keep only the thumbnail additions — no more accidental rewrites of the example code. All 10 files now contain only their original content plus the thumbnail path.

Let me know if there's anything else that needs to be addressed!

Hi @harshitnagar22, please do not use AI to write your comments except for direct translation. We would much prefer to see your thoughts without AI "polish"

@scottshambaugh Got it, and sorry about that. I’ll make sure to write the comments in my own words from now on. Please let me know if there’s anything else you’d like me to change in the PR.

The thumbnails show a representative output frame from each example.

Can you look through some of the previous discussions on thumbnails? My hang up here is that the representative output doesn't necessarily do a good job of illustrating why a person would want to look at the example. For example, this tells me nothing about font indexing:

The thumbnails show a representative output frame from each example.

Can you look through some of the previous discussions on thumbnails? My hang up here is that the representative output doesn't necessarily do a good job of illustrating why a person would want to look at the example. For example, this tells me nothing about font indexing:

Good point. I see what you mean. I’ll go through the earlier thumbnail discussions and revisit this.

@story645 — thanks for the feedback! I've updated the weak thumbnails for font_indexing, frame_grabbing, and multiprocess to better illustrate what each example does. Let me know if these work better or if you'd like any further adjustments.

Please make sure images are minified.

We serve a lot of thumbnails on the examples page and don’t want to create more traffic than necessary.

Use optipng, which is used internally for the rendered example plots, or manually minify through a separate tool such as https://tinypng.com/

The Lasso selector image should have a visible lasso

Generally, it would be good to have a way to recreate/modify the images later. Please consider how that could be supported, e.g. through a short description/script as a comment in the example sources?

@timhoffm Thanks for the feedback! I took another pass over the thumbnails and made a few updates:

• Updated the lasso_selector_demo thumbnail to better show the interactive selection.
• Added short description to the relevant examples.
• Minified all 9 PNG thumbnails to keep their file sizes smaller.

Let me know if these work better or if you'd like any further adjustments.

Hi @timhoffm and @story645, I just pushed another round of updates.

Whenever you have a chance, I'd love to hear your thoughts. If there's anything else that you'd like me to tweak, I'm happy to do it.

Whenever you have a chance, I'd love to hear your thoughts

I think the images are still a bit disconnected from the point of the figure and adding extraneous information (what does kerning have to do w/ font index? what about multiprocessing shows multiprocessing?) And the figures lack the scripts/prompts to reproduce them.