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

README animation gif example is now up to date #181

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

GenevieveBuckley
Copy link
Contributor

I noticed the animation screencast on the README page had some deprecated code visible in the gif, so I've updated it.

Now the README has:

  • A complete code snippet you can copy-paste and run immediately
  • A cropped version of the gif, that doesn't show the notebook cell with the old & deprecated animate_glpyhs command, which has been superseded by animation_control

Here's a static screenshot of the revised section:

screen shot 2018-09-26 at 9 29 07 pm

@maartenbreddels
Copy link
Collaborator

That is super nice!!!
Could you maybe put the gif in this issue:
#15
so we don't have to put the .gif in the repo (otherwise it gets super large)

@GenevieveBuckley
Copy link
Contributor Author

Done. Is issue #15 the right place to put the other two gifs in misc/ as well?

@maartenbreddels
Copy link
Collaborator

yeah, put any gif you want there, it's better for not growing the repo too much.

@maartenbreddels maartenbreddels force-pushed the master branch 2 times, most recently from e861e87 to 6dead3f Compare April 13, 2021 12:20
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.

2 participants