animation helpers#1229
Merged
Merged
Conversation
for more information, see https://pre-commit.ci
more documentation (still unfinished) tweak some argument handling
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| from ._helpers import * | ||
| from ._quick import * | ||
| from ._interact import * | ||
| from ._quick import * |
Check notice
Code scanning / CodeQL
'import *' may pollute namespace Note
for more information, see https://pre-commit.ci
replace this wrapper with a flag
for more information, see https://pre-commit.ci
ddkohler
requested review from
darienmorrow,
kameyer226,
ksunden,
rpm4 and
untzag
as code owners
pyplot needs to manage the plots for animations
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
remove initial blank figure calculated nfigs handles reduced channel size (that is not caused by constants)
for more information, see https://pre-commit.ci
Contributor
Author
|
It looks like this is far enough along that we can merge now and deal with issues as they come. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
The intent here is to wrap
matplotlib.animations.FuncAnimateso that it can handle data objects, interact2D figures, or quick[1/2]D-formatted figures and produce animations.To make this work with quick plotters, a significant refactor was done.
Added
animate.animate2D: versatile conversion of data to animation; lots of matplotlib customizationsanimate.animate_interact2D: create an animation from an interact2D object. The animation walks all the sliders.animate_quick: create an animation whose frames are the figures that would be created in a quick1D or quick2D callQuick2DIterator,Quick1DIterator: iterator classes for making figures for each object in a data chop.quick2Ds,quick1Ds: functions that generate the quick iterator classes using explicit arguments.Changed
interact2Dhas a static output type (rather than a SimpleNamespace instance) to clearly convey the inputs for animate_interact2D.quick1D,quick2D: refactored for integration with iterators, animations (these functions are wrappers for the classQuick1DLegacy,Quick2DLegacy).TODO
wt5 animateentry pointincorporate snaking (for 4+ dimensions)discarding as a low priorityanimate2D: options for plotting something other than the final two axesartists.quick(minimize redundancy between iterator- and list-based functions)Checklist