The example does not reset the plotted data when the animation is rerun/resaved after having been running/saved once. To solve this I added an init function to resets the animation.
The init function is only called at construction time and not when restarting the animation so the above doesn't work. I adapted the code to make sure that init is called at both the beginning of a save sequence and when showing the animation.
When saving the figure more than once or showing and saving the animation the second version of the animation contains wrong frames (the first frame at the end or missing frames in the beginning an so on) This is caused by the animation iterating over_save_seq and updating_save_seq while iterating. I have fixed this by copying_save_seq before iterating.

@dopplershift does this make sense or do you see a better way of fixing these issues?

jenshnielsen added3 commits

July 27, 2015 21:56

Make sure that animations are probably cleared before rerunning by ca…

6f674d5

…lling init both when showing and saving

Make a copy of _save_seq to iterate over

2b77d8e

Add init function to anim decay example

aa04769

Ensures that the plot is cleared before restarting the animation

tacaswell added the status: needs review label

Jul 27, 2015

Copy link

Contributor

dopplershift commentedJul 27, 2015

Looks good to me. Just waiting on Travis to confirm all good.

PEP8

b4b0c91

tacaswell added this to thenext point release milestone

Jul 28, 2015

tacaswell added a commit that referenced this pull request

Jul 31, 2015

Merge pull request#4800from jenshnielsen/funcanimfixes

5c58ea9

FIX: Fixes to funcanimation

tacaswell merged commit5c58ea9 intomatplotlib:master

Jul 31, 2015

tacaswell removed the status: needs review label

Jul 31, 2015

dopplershift added a commit to dopplershift/matplotlib that referenced this pull request

Aug 4, 2015

Fix infinite recursion.

6d52c41

This was introduced bymatplotlib#4800. We need to disconnect the initial drawcallback in _start() before doing _init_draw(), otherwise, we end upback in _start() when the draw_event fires.Also go ahead and remove the call to _init_draw() in __init__(), since_start() will handle it.

jenshnielsen deleted the funcanimfixes branch

August 20, 2015 12:42

tacaswell pushed a commit to tacaswell/matplotlib that referenced this pull request

Aug 27, 2015

Fix infinite recursion.

f25acff

This was introduced bymatplotlib#4800. We need to disconnect the initial drawcallback in _start() before doing _init_draw(), otherwise, we end upback in _start() when the draw_event fires.Also go ahead and remove the call to _init_draw() in __init__(), since_start() will handle it.

This was referencedNov 4, 2015

FuncAnimation w/o init_func breaks when saving#5399

Closed

Only call init function at the begining of a save when user supplied#5402

Merged