diff mbox

[FFmpeg-devel] Fate/exr : add test for PXR24 with half and uint32

Message ID CAJiLW2MY4iawMarq65rbd8frbgd4060_iPJj=8ymEs+ouBuX=A@mail.gmail.com
State Accepted
Headers show

Commit Message

Martin Vignali Nov. 17, 2016, 8:35 p.m. UTC
Hello,

in attach a patch to add a fate test for pxr24 with half and uint32 layer

Need to be apply after the patch in discussion :
libavcodec/exr : add support for uint32 channel decoding with pxr24


Relative to discussions about the crash with half/uint32 exr file
exr: limit expected_len to tmp buffer size
libavcodec/exr : Fix channel size calc


Sample can be found here and need to be put inside ./fate-suite/exr
https://we.tl/ULGDVxQXGy


Martin

Comments

Michael Niedermayer Nov. 18, 2016, 2 a.m. UTC | #1
On Thu, Nov 17, 2016 at 09:35:03PM +0100, Martin Vignali wrote:
> Hello,
> 
> in attach a patch to add a fate test for pxr24 with half and uint32 layer
> 
> Need to be apply after the patch in discussion :
> libavcodec/exr : add support for uint32 channel decoding with pxr24
> 
> 
> Relative to discussions about the crash with half/uint32 exr file
> exr: limit expected_len to tmp buffer size
> libavcodec/exr : Fix channel size calc
> 
> 

> Sample can be found here and need to be put inside ./fate-suite/exr
> https://we.tl/ULGDVxQXGy

uploaded

[...]
Andreas Cadhalpun Nov. 25, 2016, 12:03 a.m. UTC | #2
On 17.11.2016 21:35, Martin Vignali wrote:
> From 50d523d579cd8303aaab3bd7105b28359d81d13f Mon Sep 17 00:00:00 2001
> From: Martin Vignali <martin.vignali@gmail.com>
> Date: Thu, 17 Nov 2016 21:25:19 +0100
> Subject: [PATCH 3/3] fate/exr : add test for prx24 with half and uint32
>  channel
> 
> ---
>  tests/fate/image.mak                                   | 3 +++
>  tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9 | 6 ++++++
>  2 files changed, 9 insertions(+)
>  create mode 100644 tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9

Applied.

Best regards,
Andreas
diff mbox

Patch

From 50d523d579cd8303aaab3bd7105b28359d81d13f Mon Sep 17 00:00:00 2001
From: Martin Vignali <martin.vignali@gmail.com>
Date: Thu, 17 Nov 2016 21:25:19 +0100
Subject: [PATCH 3/3] fate/exr : add test for prx24 with half and uint32
 channel

---
 tests/fate/image.mak                                   | 3 +++
 tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9 | 6 ++++++
 2 files changed, 9 insertions(+)
 create mode 100644 tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9

diff --git a/tests/fate/image.mak b/tests/fate/image.mak
index c19312a..1671131 100644
--- a/tests/fate/image.mak
+++ b/tests/fate/image.mak
@@ -220,6 +220,9 @@  fate-exr-rgb-scanline-pxr24-float-half-l1: CMD = framecrc -i $(TARGET_SAMPLES)/e
 FATE_EXR += fate-exr-rgb-scanline-pxr24-float-half-l2
 fate-exr-rgb-scanline-pxr24-float-half-l2: CMD = framecrc -layer "VRaySamplerInfo" -i $(TARGET_SAMPLES)/exr/rgb_scanline_pxr24_float_half.exr -pix_fmt rgba64le
 
+FATE_EXR += fate-exr-rgb-scanline-pxr24-half-uint32-13x9
+fate-exr-rgb-scanline-pxr24-half-uint32-13x9: CMD = framecrc -i $(TARGET_SAMPLES)/exr/rgb_scanline_pxr24_half_uint32_13x9.exr -pix_fmt rgba64le
+
 FATE_EXR += fate-exr-rgb-scanline-zip-half-float-l1
 fate-exr-rgb-scanline-zip-half-float-l1: CMD = framecrc -i $(TARGET_SAMPLES)/exr/rgb_scanline_zip_half_float.exr -pix_fmt rgb48le
 
diff --git a/tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9 b/tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9
new file mode 100644
index 0000000..4a54c33
--- /dev/null
+++ b/tests/ref/fate/exr-rgb-scanline-pxr24-half-uint32-13x9
@@ -0,0 +1,6 @@ 
+#tb 0: 1/25
+#media_type 0: video
+#codec_id 0: rawvideo
+#dimensions 0: 13x9
+#sar 0: 0/1
+0,          0,          0,        1,      936, 0x7b44246e
-- 
1.9.3 (Apple Git-50)