[FFmpeg-devel,RFC] fate/cineform : add test for yuv 10b

Submitted by Carl Eugen Hoyos on Oct. 1, 2017, 8:04 p.m.

Details

Message ID CAB0OVGo78nN+UTEL7XkxLoP0mSyDYP7q2Ym6ADGDasG-P0YoVQ@mail.gmail.com
State New
Headers show

Commit Message

Carl Eugen Hoyos Oct. 1, 2017, 8:04 p.m.
2017-09-25 11:46 GMT+02:00 Martin Vignali <martin.vignali@gmail.com>:
>>
>>
>> Yes.
>>
>> Or we could switch now to a file that we know we will have
>> to add.
>> No strong opinion here: I just wanted to point out that we
>> will need another sample - that we already have. I wonder
>> if keeping the size of fate small(er) is also a relevant
>> argument.
>>
>
> Ok,
> feel free to replace the file, and send a new fate test patch

Alternative patch attached.
File in http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket6675/
Current file size: 1244380
New file: 2892690

The new file shows a visual issue with the current decoder.

Carl Eugen

Patch hide | download patch | download mbox

From aabf425e19247ad0be3d667842fc81f63ae6ea3b Mon Sep 17 00:00:00 2001
From: Martin Vignali <martin.vignali@gmail.com>
Date: Sun, 1 Oct 2017 21:57:54 +0200
Subject: [PATCH] fate: Add test for cineform yuv 10b.

---
 tests/Makefile                    |    1 +
 tests/fate/cineform.mak           |    5 +++++
 tests/ref/fate/cineform-yuv10b-hd |    6 ++++++
 3 files changed, 12 insertions(+)
 create mode 100644 tests/fate/cineform.mak
 create mode 100644 tests/ref/fate/cineform-yuv10b-hd

diff --git a/tests/Makefile b/tests/Makefile
index 99f7e17..67d14dc 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -116,6 +116,7 @@  include $(SRC_PATH)/tests/fate/bmp.mak
 include $(SRC_PATH)/tests/fate/canopus.mak
 include $(SRC_PATH)/tests/fate/cdxl.mak
 include $(SRC_PATH)/tests/fate/checkasm.mak
+include $(SRC_PATH)/tests/fate/cineform.mak
 include $(SRC_PATH)/tests/fate/concatdec.mak
 include $(SRC_PATH)/tests/fate/cover-art.mak
 include $(SRC_PATH)/tests/fate/dca.mak
diff --git a/tests/fate/cineform.mak b/tests/fate/cineform.mak
new file mode 100644
index 0000000..ca65eef
--- /dev/null
+++ b/tests/fate/cineform.mak
@@ -0,0 +1,5 @@ 
+FATE_CINEFORM += fate-cineform-yuv10b-hd
+fate-cineform-yuv10b-hd: CMD = framecrc -i $(TARGET_SAMPLES)/cineform/cineform_colour.mov -pix_fmt yuv422p10le -an
+
+FATE_SAMPLES_AVCONV-$(call DEMDEC, MOV, CFHD) += $(FATE_CINEFORM)
+fate-cineform: $(FATE_CINEFORM)
diff --git a/tests/ref/fate/cineform-yuv10b-hd b/tests/ref/fate/cineform-yuv10b-hd
new file mode 100644
index 0000000..200f504
--- /dev/null
+++ b/tests/ref/fate/cineform-yuv10b-hd
@@ -0,0 +1,6 @@ 
+#tb 0: 1/30
+#media_type 0: video
+#codec_id 0: rawvideo
+#dimensions 0: 1920x1080
+#sar 0: 1/1
+0,          0,          0,        1,  8294400, 0x46ef6c43
-- 
1.7.10.4