-
Notifications
You must be signed in to change notification settings - Fork 207
ENH: Use broader def of Animation #693
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #693 +/- ##
==========================================
- Coverage 97.81% 97.59% -0.22%
==========================================
Files 32 32
Lines 3429 3495 +66
==========================================
+ Hits 3354 3411 +57
- Misses 75 84 +9
Continue to review full report at Codecov.
|
@larsoner how did you know the error:
was caused by optipng? That error message is a bit cryptic. |
Warnings are treated as errors, I ctrl-F'ed "warning" in the doc build log and found the warning |
I see the |
Yeah. Newer sphinx is supposed to tell you that there was a warning that was treated as an error that killed the build but apparently it didn't work in this case |
Ah right. Thanks!
…On Thu, 21 May 2020, 22:33 Eric Larson ***@***.***> wrote:
Does it just mean that there was a warning somewhere?
Yeah. Newer sphinx is supposed to tell you that there was a warning that
was treated as an error that killed the build but apparently it didn't work
in this case
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#693 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFQ333LBGY6N4H5OQXQZ7RDRSWFZLANCNFSM4NHCLTSQ>
.
|
Matplotlib also has
ArtistAnimation
. It would be nice to have this for matplotlib (matplotlib/matplotlib#17477), maybe in a 0.7.1 release sometime? Let's wait to see what else breaks in the meantime before cutting one, though.