Skip to content

Fix screenshots for Sphinx Gallery#782

Merged
banesullivan merged 1 commit intorelease/0.25from
hotfix/patch_screenshots
Jun 6, 2020
Merged

Fix screenshots for Sphinx Gallery#782
banesullivan merged 1 commit intorelease/0.25from
hotfix/patch_screenshots

Conversation

@akaszynski
Copy link
Copy Markdown
Member

@akaszynski akaszynski commented Jun 5, 2020

Many projects use pyvista for sphinx_gallery and this PR seeks to keep the past behavior without sacrificing performance by unnecessarily always saving screenshots.

On initialization, pyvista checks if sphinx has been imported. If it has, screenshots are automatically saved upon the closing of a plot.

Follow up to #730

@akaszynski akaszynski requested a review from banesullivan June 5, 2020 22:14
@banesullivan
Copy link
Copy Markdown
Member

This is a clever fix! Let me think about this one a bit more before we merge and release. Thanks for getting to this so quickly, @akaszynski!

Copy link
Copy Markdown
Member

@banesullivan banesullivan left a comment

Choose a reason for hiding this comment

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

Need a fix ASAP for the Tranfrom2020 hackathons. Merging and releasing

@banesullivan
Copy link
Copy Markdown
Member

Actually. I am going to document this real fast

@banesullivan banesullivan added bug-fix priority-super-duper-high GO GET SOME COFFEE & PLEASE TRY TO HELP FIX THIS ASAP labels Jun 6, 2020
@banesullivan banesullivan merged commit 50f3fbf into release/0.25 Jun 6, 2020
@akaszynski akaszynski deleted the hotfix/patch_screenshots branch November 10, 2020 19:53
@banesullivan banesullivan added the bug Uh-oh! Something isn't working as expected. label Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Uh-oh! Something isn't working as expected. priority-super-duper-high GO GET SOME COFFEE & PLEASE TRY TO HELP FIX THIS ASAP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants