Skip to content

DOC MEP12 - converted animation to SG/MEP12 compatible #7329

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

Merged
merged 2 commits into from
Oct 31, 2016
Merged

DOC MEP12 - converted animation to SG/MEP12 compatible #7329

merged 2 commits into from
Oct 31, 2016

Conversation

NelleV
Copy link
Member

@NelleV NelleV commented Oct 21, 2016

Minor modification to have part of the animation section Sphinx-gallery compatible.

I'm pretty sure there is a problem in the Baye's update example, considering it does not converge to the theoretical value the code mentions…

Supersedes: #6486

See #7206

@phobson phobson added Documentation MEP: MEP12 gallery and examples improvements labels Oct 22, 2016
Copy link
Member

@phobson phobson left a comment

Choose a reason for hiding this comment

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

These look good to me. Thanks to @NelleV and @adasilva for the time put into this!

@phobson phobson changed the title [MRG] DOC MEP12 - converted animation to SG/MEP12 compatible [MRG+1] DOC MEP12 - converted animation to SG/MEP12 compatible Oct 23, 2016
t += 0.1
yield t, np.sin(2*np.pi*t) * np.exp(-t/10.)


def init():
"""Create a line
Copy link
Member

Choose a reason for hiding this comment

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

This is not quite correct. It doesn't create the line, it updates the line's data with the initial value, which is maybe a bit odd, I guess.

@@ -1,17 +1,31 @@
"""
=====
Decay
Copy link
Member

Choose a reason for hiding this comment

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

Probably needs another word, "Decay animation", or something.

Simple animation examples
=========================

This example contains two animations. The first is a random walk plot. the
Copy link
Member

Choose a reason for hiding this comment

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

the -> The

============
Oscilloscope
============

Emulate an oscilloscope. Requires the animation API introduced in
matplotlib 1.0 SVN.
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this information any more?

@tacaswell tacaswell merged commit 00ae164 into matplotlib:master Oct 31, 2016
tacaswell added a commit that referenced this pull request Oct 31, 2016
DOC: converted animation to SG/MEP12 compatible
@tacaswell
Copy link
Member

Thanks

Backported to v2.x as 66941ec

@QuLogic QuLogic added this to the 2.0.1 (next bug fix release) milestone Oct 31, 2016
@QuLogic QuLogic changed the title [MRG+1] DOC MEP12 - converted animation to SG/MEP12 compatible DOC MEP12 - converted animation to SG/MEP12 compatible Nov 24, 2016
@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation MEP: MEP12 gallery and examples improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants