NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.4k

Test doc build time on CI without sphinx-gallery#29493

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Closed

timhoffm wants to merge1 commit intomatplotlib:mainfromtimhoffm:doc-test-no-sphinx-gallery

Closed

Test doc build time on CI without sphinx-gallery#29493

timhoffm wants to merge1 commit intomatplotlib:mainfromtimhoffm:doc-test-no-sphinx-gallery

Conversation

Copy link

Member

timhoffm commentedJan 20, 2025

DO NOT MERGE. This is a test to see how much time we spent on running the gallery examples. See#22956 (comment)

Test doc build time on CI without sphinx-gallery

45e6a97

DO NOT MERGE. This is a test to see how much time we spent on running the gallery examples.Seematplotlib#22956 (comment)

Copy link

MemberAuthor

timhoffm commentedJan 20, 2025•
edited
Loading

Result: the pure build time was 4min compared to 7.5-8min previously. So there's a potential saving of ~3.5-4min. However, such build still produces warnings (currently 54) in the form of

/home/circleci/project/doc/users/prev_whats_new/whats_new_1.0.rst:29: WARNING: Could not obtain image size. :scale: option is ignored./home/circleci/project/doc/users/prev_whats_new/whats_new_1.0.rst:29: WARNING: Cannot scale image!

because we reuse figures from the gallery via the.. figure:: directive such as

matplotlib/doc/users/prev_whats_new/whats_new_1.0.rst

Lines 29 to 32 in06548d2