[FFmpeg-devel] [PATCH 6/6] lavc/cbs: Add tests for VP9

[Mark Thompson]

On 01/05/18 18:37, James Almer wrote:
> On 4/30/2018 8:26 PM, Mark Thompson wrote:
>> Uses the same mechanism as other codecs - conformance test files are
>> passed through the metadata filter (which, with no options, reads the
>> input and writes it back) and the output verified to match the input.
>> ---
>>  tests/fate/cbs.mak                                 | 34 ++++++++++++++++++----
>>  tests/ref/fate/cbs-vp9-vp90-2-03-deltaq            |  1 +
>>  tests/ref/fate/cbs-vp9-vp90-2-05-resize            |  1 +
>>  tests/ref/fate/cbs-vp9-vp90-2-06-bilinear          |  1 +
>>  tests/ref/fate/cbs-vp9-vp90-2-09-lf_deltas         |  1 +
>>  .../ref/fate/cbs-vp9-vp90-2-10-show-existing-frame |  1 +
>>  .../fate/cbs-vp9-vp90-2-10-show-existing-frame2    |  1 +
>>  .../ref/fate/cbs-vp9-vp90-2-segmentation-aq-akiyo  |  1 +
>>  .../ref/fate/cbs-vp9-vp90-2-segmentation-sf-akiyo  |  1 +
>>  tests/ref/fate/cbs-vp9-vp90-2-tiling-pedestrian    |  1 +
>>  tests/ref/fate/cbs-vp9-vp91-2-04-yuv440            |  1 +
>>  tests/ref/fate/cbs-vp9-vp91-2-04-yuv444            |  1 +
>>  tests/ref/fate/cbs-vp9-vp92-2-20-10bit-yuv420      |  1 +
>>  tests/ref/fate/cbs-vp9-vp93-2-20-10bit-yuv422      |  1 +
>>  tests/ref/fate/cbs-vp9-vp93-2-20-12bit-yuv444      |  1 +
>>  15 files changed, 43 insertions(+), 5 deletions(-)
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-03-deltaq
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-05-resize
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-06-bilinear
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-09-lf_deltas
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-10-show-existing-frame
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-10-show-existing-frame2
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-segmentation-aq-akiyo
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-segmentation-sf-akiyo
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp90-2-tiling-pedestrian
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp91-2-04-yuv440
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp91-2-04-yuv444
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp92-2-20-10bit-yuv420
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp93-2-20-10bit-yuv422
>>  create mode 100644 tests/ref/fate/cbs-vp9-vp93-2-20-12bit-yuv444
> 
> LGTM.
> 
> Unrelated to this patch, but could you add -y to the command line for
> all cbs tests? That way it will not fail/abort if the output file
> already exists (Something that can happen if the test failed previously
> as no cleanup takes place in that case).

Right, yes - I've added this as a further trivial patch and applied the rest of the set (subscripts + VP9).

Thank you for reviewing all of that,

- Mark

(AV1 support to follow at some point.)