Skip to content

Conversation

@gully
Copy link
Owner

@gully gully commented Sep 11, 2023

Really helpful for visualizing the state change

@gully gully linked an issue Sep 11, 2023 that may be closed by this pull request
@gully
Copy link
Owner Author

gully commented Sep 11, 2023

BlaseScene.mp4

@gully
Copy link
Owner Author

gully commented Sep 11, 2023

Ok, I experimented with this .animate() method, and while it's really cool, I think we should not support it. Here's why:

The code structure would require an inner class.

While feasible, the code would just be a little clunky and tricky to debug.

Folks will want more customizations than can be anticipated

Inevitably folks will want more customizations than can be anticipated, since the choice of sampling is very problem specific owing to the high dynamic range of the spectrum.

So I suggest we simply terminate this PR, and instead add a tutorial showing how to combine with manim by hand. Folks will be able to customize that to their heart's content.

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.

Make an animation of how blasé works

2 participants