You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #708 relaxed the requirement at the C++ level that the number of frames and the duration of the video or audio file must be present. We are doing this in eventual support of live streaming (see #695). The tests pass, and we checked that when we hack the C++ code to remove obtaining the values, we don't run into any segfaults. But, we don't have any tests that ensure the correct behavior with the values missing.
We should:
Check in a small test video that has missing metadata for number of frames and duration.
Use it for some of our ops tests - in particular, range queries in both indices and time. Tests that use the PythonVideoDeocder will always fail because we assert the presence of these values at the Python level.