[FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

[FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

[Mark Thompson]

Bitstream generated using the reference encoder, then edited to fix the
colour description and an extra metadata block added.  FFmpeg decoder
output is identical to the reference decoder output.

The content used is the first three frames of "Waterfall" from the SVT
Open Content Video Test Suite 2022.  This is copyright Sveriges
Television AB and is used under the Creative Commons Attribution 4.0
International License.
---
Required sample is at <https://0x0.st/8WNx.apv>.  Please upload as "fate/apv/profile_422-10.apv" (new directory).

Currently the reference encoder only generates 422-10 profile output (profile_idc == 33).  The output stream looks right other than the profile_idc value for other profiles such as 444-10 or 400-10, but I think it is better to wait until they are officially supported before generating a reference sample.

Thanks,

- Mark

 tests/Makefile            | 1 +
 tests/fate/apv.mak        | 7 +++++++
 tests/ref/fate/apv-422-10 | 8 ++++++++
 3 files changed, 16 insertions(+)
 create mode 100644 tests/fate/apv.mak
 create mode 100644 tests/ref/fate/apv-422-10

diff --git a/tests/Makefile b/tests/Makefile
index 0c08f68713..68e68f5241 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -156,6 +156,7 @@ include $(SRC_PATH)/tests/fate/amrnb.mak
 include $(SRC_PATH)/tests/fate/amrwb.mak
 include $(SRC_PATH)/tests/fate/api.mak
 include $(SRC_PATH)/tests/fate/apng.mak
+include $(SRC_PATH)/tests/fate/apv.mak
 include $(SRC_PATH)/tests/fate/atrac.mak
 include $(SRC_PATH)/tests/fate/audio.mak
 include $(SRC_PATH)/tests/fate/bmp.mak
diff --git a/tests/fate/apv.mak b/tests/fate/apv.mak
new file mode 100644
index 0000000000..37e0b9f7be
--- /dev/null
+++ b/tests/fate/apv.mak
@@ -0,0 +1,7 @@
+FATE_APV = fate-apv-422-10
+
+FATE_SAMPLES_FFMPEG-$(call FRAMECRC, APV, APV, SCALE_FILTER) += $(FATE_APV)
+
+fate-apv:	$(FATE_APV)
+
+fate-apv-422-10:	CMD = framecrc -i $(TARGET_SAMPLES)/apv/profile_422-10.apv -pix_fmt yuv422p10le -vf scale
diff --git a/tests/ref/fate/apv-422-10 b/tests/ref/fate/apv-422-10
new file mode 100644
index 0000000000..d000751207
--- /dev/null
+++ b/tests/ref/fate/apv-422-10
@@ -0,0 +1,8 @@
+#tb 0: 1/30
+#media_type 0: video
+#codec_id 0: rawvideo
+#dimensions 0: 320x180
+#sar 0: 1/1
+0,          0,          0,        1,   230400, 0x07f1e56d
+0,          1,          1,        1,   230400, 0x0bd1c913
+0,          2,          2,        1,   230400, 0xefd02824
-- 
2.47.2

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

[James Almer]

[-- Attachment #1.1.1: Type: text/plain, Size: 2683 bytes --]

On 4/27/2025 3:13 PM, Mark Thompson wrote:
> Bitstream generated using the reference encoder, then edited to fix the
> colour description and an extra metadata block added.  FFmpeg decoder
> output is identical to the reference decoder output.
> 
> The content used is the first three frames of "Waterfall" from the SVT
> Open Content Video Test Suite 2022.  This is copyright Sveriges
> Television AB and is used under the Creative Commons Attribution 4.0
> International License.
> ---
> Required sample is at <https://0x0.st/8WNx.apv>.  Please upload as "fate/apv/profile_422-10.apv" (new directory).

Uploaded.

> 
> Currently the reference encoder only generates 422-10 profile output (profile_idc == 33).  The output stream looks right other than the profile_idc value for other profiles such as 444-10 or 400-10, but I think it is better to wait until they are officially supported before generating a reference sample.
> 
> Thanks,
> 
> - Mark
> 
>   tests/Makefile            | 1 +
>   tests/fate/apv.mak        | 7 +++++++
>   tests/ref/fate/apv-422-10 | 8 ++++++++
>   3 files changed, 16 insertions(+)
>   create mode 100644 tests/fate/apv.mak
>   create mode 100644 tests/ref/fate/apv-422-10
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index 0c08f68713..68e68f5241 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -156,6 +156,7 @@ include $(SRC_PATH)/tests/fate/amrnb.mak
>   include $(SRC_PATH)/tests/fate/amrwb.mak
>   include $(SRC_PATH)/tests/fate/api.mak
>   include $(SRC_PATH)/tests/fate/apng.mak
> +include $(SRC_PATH)/tests/fate/apv.mak
>   include $(SRC_PATH)/tests/fate/atrac.mak
>   include $(SRC_PATH)/tests/fate/audio.mak
>   include $(SRC_PATH)/tests/fate/bmp.mak
> diff --git a/tests/fate/apv.mak b/tests/fate/apv.mak
> new file mode 100644
> index 0000000000..37e0b9f7be
> --- /dev/null
> +++ b/tests/fate/apv.mak
> @@ -0,0 +1,7 @@
> +FATE_APV = fate-apv-422-10
> +
> +FATE_SAMPLES_FFMPEG-$(call FRAMECRC, APV, APV, SCALE_FILTER) += $(FATE_APV)
> +
> +fate-apv:	$(FATE_APV)
> +
> +fate-apv-422-10:	CMD = framecrc -i $(TARGET_SAMPLES)/apv/profile_422-10.apv -pix_fmt yuv422p10le -vf scale
> diff --git a/tests/ref/fate/apv-422-10 b/tests/ref/fate/apv-422-10
> new file mode 100644
> index 0000000000..d000751207
> --- /dev/null
> +++ b/tests/ref/fate/apv-422-10
> @@ -0,0 +1,8 @@
> +#tb 0: 1/30
> +#media_type 0: video
> +#codec_id 0: rawvideo
> +#dimensions 0: 320x180
> +#sar 0: 1/1
> +0,          0,          0,        1,   230400, 0x07f1e56d
> +0,          1,          1,        1,   230400, 0x0bd1c913
> +0,          2,          2,        1,   230400, 0xefd02824


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] fate: Add test for APV 422-10 profile

[Mark Thompson]

On 27/04/2025 20:58, James Almer wrote:
> On 4/27/2025 3:13 PM, Mark Thompson wrote:
>> Bitstream generated using the reference encoder, then edited to fix the
>> colour description and an extra metadata block added.  FFmpeg decoder
>> output is identical to the reference decoder output.
>>
>> The content used is the first three frames of "Waterfall" from the SVT
>> Open Content Video Test Suite 2022.  This is copyright Sveriges
>> Television AB and is used under the Creative Commons Attribution 4.0
>> International License.
>> ---
>> Required sample is at <https://0x0.st/8WNx.apv>.  Please upload as "fate/apv/profile_422-10.apv" (new directory).
> 
> Uploaded.

And pushed the test using it.

Thanks,

- Mark

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".