pytorch / torchcodec

PyTorch video decoding
BSD 3-Clause "New" or "Revised" License
77 stars 9 forks source link

Retire testing utils `get_frame_by_name()` #304

Closed scotts closed 1 week ago

scotts commented 1 week ago

We have some Python-based tests that predate the current testing framework. The testing framework has the API call get_frame_by_name() implemented to support instances where we were testing against a reference frame and we didn't know its index.

In general, all of our Python tests should test against frames whose index we know, and we should refer to it through the APIs in our testing framework. Retiring get_frame_by_name() will required converting the existing tests to refer to the frame by index, and updating the metadata for the frame and potentially the name of the frame itself to match.