blase icon indicating copy to clipboard operation
blase copied to clipboard

[experimental] Add an emulator.animate() function using manim

Open gully opened this issue 2 years ago • 2 comments

Really helpful for visualizing the state change

gully avatar Sep 11 '23 19:09 gully

https://github.com/gully/blase/assets/860227/1c26db27-5e8e-4f29-88b6-963b189667c9

gully avatar Sep 11 '23 20:09 gully

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.

gully avatar Sep 11 '23 20:09 gully