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.) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
