Video class refactor

[timmermansjoy]

trafficstars

Description

Please include a summary of the change and which issue is fixed or implemented. Please also include relevant motivation and context (e.g. links, docs, tickets etc.).

implementing feature request: #1929

List any dependencies that are required for this change.

pyav (optional)

Type of change

Please delete options that are not relevant.

• [x] New feature (non-breaking change which adds functionality)
• [x] This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?

There will be a google colabs provided when its ready for testing as requested in #1929 and there can also be extra unit tests added if needed

Any specific deployment considerations

Documentation should be updated to give information about how the new video class will work compared to the old stand-alone functions

Docs

still need to be done

[CLAassistant]

All committers have signed the CLA.

[timmermansjoy]

@SkalskiP for your issue of #1929 Is this a good start of you feel things need to be fundamentally changed?

[SkalskiP]

Hi @timmermansjoy 👋🏻 Can you accept the CLA? We won't be able to merge this PR without it.

[timmermansjoy]

@SkalskiP i see, just did yea

[review-notebook-app[bot]]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB