Blur Animation source code and simplified output #328
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.
Hello everyone!
I've seen from this issue, that we were looking to implement a better blur animation figure for section 06-blurring.
I've implemented a Python script that creates this animation with Matplotlib. Here are a few points:
episodes/data/A.tif
episodes/fig/blur-demo.gif
env.yml
file for the requirementsA final extra point from my side: I'm a big believer that every output should be associated with its source-code, especially in the carpentries context where we are trying to keep things reproducible. In this case, I couldn't find the code used to create the previous version of the
blur-demo.gif
so I took the freedom to create aepisodes/files/source-code/
directory, where we could pool together all the code snippets for generating all the output that would later feed on the website (this is where my currentcreate_blur_animation.py
file is). I am unsure if this is the best place to have such folder, maybe in the root could even be better for clear reference.I hope this is helpful for the issue and course material. I am open to improve it in the near future after some feedback!