[FFmpeg-cvslog] fate: add R210 encoding/decoding test

[Reimar Döffinger]

On 29 Jan 2012, at 02:53, git at videolan.org (Paul B Mahol) wrote:
> ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Sun Jan 29 00:32:30 2012 +0000| [931ec4a2f64143c7029bc8699534268ae0a8946e] | committer: Michael Niedermayer
> 
> fate: add R210 encoding/decoding test
> 
> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> 
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=931ec4a2f64143c7029bc8699534268ae0a8946e
> ---
> 
> tests/codec-regression.sh |    5 +++++
> tests/ref/vsynth1/r210    |    4 ++++
> tests/ref/vsynth2/r210    |    4 ++++
> 3 files changed, 13 insertions(+), 0 deletions(-)
> 
> diff --git a/tests/codec-regression.sh b/tests/codec-regression.sh
> index 5871c2d..3a16a02 100755
> --- a/tests/codec-regression.sh
> +++ b/tests/codec-regression.sh
> @@ -319,6 +319,11 @@ do_video_encoding rgb.avi "-an -vcodec rawvideo -pix_fmt bgr24"
> do_video_decoding "" "-pix_fmt yuv420p"
> fi
> 
> +if [ -n "$do_r210" ] ; then
> +do_video_encoding r210.avi "-an -c:v r210"
> +do_video_decoding
> +fi
> +
> if [ -n "$do_v210" ] ; then
> do_video_encoding v210.avi "-an -c:v v210"
> do_video_decoding "" "-pix_fmt yuv420p"
> diff --git a/tests/ref/vsynth1/r210 b/tests/ref/vsynth1/r210
> new file mode 100644
> index 0000000..02babd0
> --- /dev/null
> +++ b/tests/ref/vsynth1/r210
> @@ -0,0 +1,4 @@
> +604a34c9f9151891c2d63d2652b75329 *./tests/data/vsynth1/r210.avi
> + 22125260 ./tests/data/vsynth1/r210.avi
> +1ac5ec3bbb6bfbdf1ce6316e48e5fd89 *./tests/data/r210.vsynth1.out.yuv
> +stddev:   91.79 PSNR:  8.87 MAXDIFF:  254 bytes: 30412800/  7603200

This test is clearly wrong, it compares two files with a 3x size difference.
You should add a pix_fmt to make it convert back to the original format at the end.
Note that this means the lowest 2 bits will not be tested much, if you want to change that we'd have to generate > 8 bit reference files.