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

Updated Lecture-4-Matplotlib.ipynb: Syntax, FFMpeg workaround, clarification #12

Merged
merged 1 commit into from Dec 11, 2013

Conversation

westurner
Copy link
Contributor

No description provided.

@jrjohansson
Copy link
Owner

Thanks for the improvements! I will merge, but it seems that in your updated notebook the animation example does not work. Any ideas why?

http://nbviewer.ipython.org/github/westurner/scientific-python-lectures/blob/ad71d7e463bba7ab5e941bd5d497422e729eb591/Lecture-4-Matplotlib.ipynb

@westurner
Copy link
Contributor Author

Thank you!

Sorry, I'm not sure - I assumed it was my browser and checked the .mp4 with VLC.

I added the explicit writer because of a traceback that referenced 'saving' which lead to a post stack overflow that suggested a bug in matplotlib which I haven't yet had a chance to track down.

@westurner
Copy link
Contributor Author

@jrjohansson
Copy link
Owner

Yes, there seems to be some problems with matplotlib and the animation module.. I've been having trouble to get something to work on the Mac with the latest version of matplotlib, but it works fine on Linux. So I'll merge your PR now, but will revert to the previous way of calling anim.save for now so that the video can be shown in the notebook.

jrjohansson added a commit that referenced this pull request Dec 11, 2013
Updated Lecture-4-Matplotlib.ipynb: Syntax, FFMpeg workaround, clarification
@jrjohansson jrjohansson merged commit edb1e61 into jrjohansson:master Dec 11, 2013
@westurner
Copy link
Contributor Author

Good call. Thanks again!

@westurner westurner deleted the lecture_4_wording_ffmpeg branch December 11, 2013 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants