Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: pre-composite animation frames to white background #21831

Merged
merged 2 commits into from
Nov 23, 2022

Conversation

tacaswell
Copy link
Member

This fixes issues with encoding that do not support transparency and when the
Figure facecolor is transparent.

Closes #19040

PR Summary

I'm not sure how to test this as it is a visual thing is h254 encoded output.

PR Checklist

Tests and Styling

  • Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (install flake8-docstrings and run flake8 --docstring-convention=all).

Documentation

  • [n/a] New features are documented, with examples if plot related.
  • [n/a] New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).

@tacaswell tacaswell added this to the v3.6.0 milestone Dec 1, 2021
@tacaswell tacaswell marked this pull request as draft December 7, 2021 22:36
@QuLogic QuLogic modified the milestones: v3.6.0, v3.7.0 Jul 5, 2022
@rothn
Copy link

rothn commented Oct 20, 2022

Keep up the great work!

This fixes issues with encoding that do not support transparency and when the
Figure facecolor is transparent.

Closes matplotlib#19040
@tacaswell tacaswell marked this pull request as ready for review October 20, 2022 17:06
lib/matplotlib/tests/test_animation.py Outdated Show resolved Hide resolved
lib/matplotlib/animation.py Show resolved Hide resolved
We no longer just hold onto the savefig kwargs passed in from the user so
adjust test to make sure the user values go through rather than simple
dictionary equality.

Set the rcparam to cover all of the lines added

Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
@QuLogic QuLogic merged commit bdf178e into matplotlib:main Nov 23, 2022
@tacaswell tacaswell deleted the fix_transparent_animation branch October 27, 2023 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v3.3.0 Regression, Animation draws artists multiple times.
4 participants