Daniel Falk
Daniel Falk
Here is a first set of unit tests. These test mostly the interface, I intend to add more tests for the actual encoding and decoding of video. Once I have...
How should the OpenCV requirement be installed? I have added it to a `video_requires` in `setup.py` so that it is installed with install_extra `video` or `all`, but it seems like...
@noklam Do you have a Windows based environment setup? In that case, could you help me get the CI through for Windows? - Windows unit test for save to S3...
Here is the codec-container compatibility matrix as from the CI pipeline. Some codecs can be installed separately (such as e.g. h264 codec which has a bad licence preventing it from...
I have added unit tests now to get 100% coverage. @noklam It seems like it is just one unit test that fails on Windows now. Did you have a Windows...
Have you had time to look anything more on this @noklam ? Or is there anyone else on the team who could help me get it trough @datajoely ?
Thanks for finding the Windows issue @noklam ! I'll look into why the tests are not failing due to the missing credentials, then I'll rebase as squash the commits to...
Hi @merelcht. Sorry for being slow on this, some other projects took priority. It would be nice to get it merged before changes are made, when is the move planned...
@noklam That's a good idea to reduce the size of the test matrix. I have removed the inheritence of the versioned dataset now. I have also tested the dataset in...
@merelcht re: _One thing I don't fully understand is the purpose of all the different video classes. As a user do I need to specify if I want to use...