Switch to imageio-ffmpeg

[jkterry1]

Per discussion in PR #1893

[jkterry1]

@zlig does your PR I just merged close this?

[zlig]

I would actually defer to @christopherhesse as it was his original suggestion.

We need to confirm if the intention was to fully switch everything to imageio-ffmpeg, which means there is more work to do, or just add it as an alternative, which means we are done as we did in #1893

[jkterry1]

Christopher Hesse isn't really involved in the maintenance. I'm happy to close this given us supporting both, however a concern that's raised is that the "other" extra in setup.py doesn't include ffmpeg, meaning this alternative rendering method isn't tested. Would you be willing to add a PR that does this?

[zlig]

The missing imageio_ffmpeg component in setup.py would not be exercised with the current build workflows though, would it?

My understanding is we need to run tests via the workflows for various systems with ffmpeg, other with avconv, and finally one for the alternative we just added with imageio-ffmpeg.