diff mbox

[FFmpeg-devel] fate: add a test for mpeg2 issue of ticket #6677

Message ID 20171023071812.32240-1-zhong.li@intel.com
State Accepted
Commit 7e294a1f09a2b7bbba5c057e23e533fffef5336d
Headers show

Commit Message

Zhong Li Oct. 23, 2017, 7:18 a.m. UTC
Signed-off-by: Zhong Li <zhong.li@intel.com>
---
 tests/fate/video.mak            |  3 +++
 tests/ref/fate/mpeg2-ticket6677 | 12 ++++++++++++
 2 files changed, 15 insertions(+)
 create mode 100644 tests/ref/fate/mpeg2-ticket6677

Comments

Michael Niedermayer Oct. 24, 2017, 9:50 p.m. UTC | #1
On Mon, Oct 23, 2017 at 03:18:12PM +0800, Zhong Li wrote:
> Signed-off-by: Zhong Li <zhong.li@intel.com>
> ---
>  tests/fate/video.mak            |  3 +++
>  tests/ref/fate/mpeg2-ticket6677 | 12 ++++++++++++
>  2 files changed, 15 insertions(+)
>  create mode 100644 tests/ref/fate/mpeg2-ticket6677

will apply

thanks

[...]
James Almer Oct. 25, 2017, 2:09 a.m. UTC | #2
On 10/24/2017 6:50 PM, Michael Niedermayer wrote:
> On Mon, Oct 23, 2017 at 03:18:12PM +0800, Zhong Li wrote:
>> Signed-off-by: Zhong Li <zhong.li@intel.com>
>> ---
>>  tests/fate/video.mak            |  3 +++
>>  tests/ref/fate/mpeg2-ticket6677 | 12 ++++++++++++
>>  2 files changed, 15 insertions(+)
>>  create mode 100644 tests/ref/fate/mpeg2-ticket6677
> 
> will apply
> 
> thanks

This seems to be failing on msvc (x86_32 at least), and on PowerPC Big
Endian. Both give two different sets of frame hashes.
Zhong Li Oct. 25, 2017, 2:37 a.m. UTC | #3
> -----Original Message-----

> From: ffmpeg-devel [mailto:ffmpeg-devel-bounces@ffmpeg.org] On Behalf

> Of James Almer

> Sent: Wednesday, October 25, 2017 10:09 AM

> To: ffmpeg-devel@ffmpeg.org

> Subject: Re: [FFmpeg-devel] [PATCH] fate: add a test for mpeg2 issue of

> ticket #6677

> 

> On 10/24/2017 6:50 PM, Michael Niedermayer wrote:

> > On Mon, Oct 23, 2017 at 03:18:12PM +0800, Zhong Li wrote:

> >> Signed-off-by: Zhong Li <zhong.li@intel.com>

> >> ---

> >>  tests/fate/video.mak            |  3 +++

> >>  tests/ref/fate/mpeg2-ticket6677 | 12 ++++++++++++

> >>  2 files changed, 15 insertions(+)

> >>  create mode 100644 tests/ref/fate/mpeg2-ticket6677

> >

> > will apply

> >

> > thanks

> 

> This seems to be failing on msvc (x86_32 at least), and on PowerPC Big

> Endian. Both give two different sets of frame hashes.


It should be caused by different idct ways. Will update, thanks for report this.

> _______________________________________________

> ffmpeg-devel mailing list

> ffmpeg-devel@ffmpeg.org

> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
diff mbox

Patch

diff --git a/tests/fate/video.mak b/tests/fate/video.mak
index 6f76365165..51678078a3 100644
--- a/tests/fate/video.mak
+++ b/tests/fate/video.mak
@@ -257,6 +257,9 @@  fate-mpeg2-ticket186: CMD = framecrc -flags +bitexact -idct simple -i $(TARGET_S
 FATE_VIDEO-$(call DEMDEC, MPEGPS, MPEG2VIDEO) += fate-mpeg2-ticket6024
 fate-mpeg2-ticket6024: CMD = framecrc -flags +bitexact -idct simple -flags +truncated -i $(TARGET_SAMPLES)/mpeg2/matrixbench_mpeg2.lq1.mpg -an
 
+FATE_VIDEO-$(call DEMDEC, MPEGVIDEO, MPEG2VIDEO) += fate-mpeg2-ticket6677
+fate-mpeg2-ticket6677: CMD = framecrc -vsync drop -i $(TARGET_SAMPLES)/mpeg2/sony-ct3.bs
+
 FATE_VIDEO-$(call DEMDEC, MV, MVC1) += fate-mv-mvc1
 fate-mv-mvc1: CMD = framecrc -i $(TARGET_SAMPLES)/mv/posture.mv -an -frames 25 -pix_fmt rgb555le
 
diff --git a/tests/ref/fate/mpeg2-ticket6677 b/tests/ref/fate/mpeg2-ticket6677
new file mode 100644
index 0000000000..5fc7cb2a26
--- /dev/null
+++ b/tests/ref/fate/mpeg2-ticket6677
@@ -0,0 +1,12 @@ 
+#tb 0: 1/30
+#media_type 0: video
+#codec_id 0: rawvideo
+#dimensions 0: 720x480
+#sar 0: 8/9
+0,          0,          0,        1,   518400, 0x354b6fc3
+0,          1,          1,        1,   518400, 0xd86f281b
+0,          2,          2,        1,   518400, 0xdd2e2b38
+0,          3,          3,        1,   518400, 0x589535d4
+0,          4,          4,        1,   518400, 0x3f8f02b6
+0,          5,          5,        1,   518400, 0xa81b246a
+0,          6,          6,        1,   518400, 0xb98dd9f7