Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
BUG: pass kwargs to TimedAnimation#2069
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Another thing: throughout this file, initialization functions are called directly, rather than with |
Thanks@jakevdp. We should apply this to the
I think I remember a discussion between@dopplershift and@jdh2358 regarding issues with multiple inheritance (in#1012 I think) and various 👍 from me |
@pelson: Yes, this should be backported to 1.3.x, and possibly 1.2.x (haven't yet checked its applicability there). I think merging to master and then cherry-picking the merge to the branches after is fine (rather than resubmitting PRs). As per |
BUG: pass kwargs to TimedAnimation
BUG: pass kwargs to TimedAnimation
BUG: pass kwargs to TimedAnimation
Also not that the call signature of |
A quick bug fix to pass keyword arguments on to the parent class.