[FFmpeg-devel] [PATCH 0/1] fate/jpegxl: add multiframe permuted TOC image parser

[FFmpeg-devel] [PATCH 0/1] fate/jpegxl: add multiframe permuted TOC image parser

[Leo Izen]

This patch requires a sample that hasn't been uploaded yet. It can be found
at [1] and it should be placed at jxl/orange.jxl once uploaded. It's 262k,
which is not very large, but large enough that I did not want to attach it
to this email.

[1]: https://buzo.us/O.jxl

sample sha256sum: 9288337dc0d37effd1f539d4e20083a8ffed757e486f4098121520c74e8de6f6
sample signature: https://buzo.us/A.asc

Leo Izen (1):
  fate/jpegxl: add multiframe permuted TOC image parser test

 tests/fate/jxl.mak                         |  3 +++
 tests/ref/fate/jxl-multiframe-permuted-toc | 11 +++++++++++
 2 files changed, 14 insertions(+)
 create mode 100644 tests/ref/fate/jxl-multiframe-permuted-toc

-- 
2.43.0

_______________________________________________
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".

[FFmpeg-devel] [PATCH 1/1] fate/jpegxl: add multiframe permuted TOC image parser test

[Leo Izen]

This test verifies the parser's handling of multiframe JXL files that
have an entropy-encoded permuted table of contents for each frame. The
testcase is actually six JXL codestreams concatenated together, and the
parser needs to be able to find the boundaries.

Signed-off-by: Leo Izen <leo.izen@gmail.com>
---
 tests/fate/jxl.mak                         |  3 +++
 tests/ref/fate/jxl-multiframe-permuted-toc | 11 +++++++++++
 2 files changed, 14 insertions(+)
 create mode 100644 tests/ref/fate/jxl-multiframe-permuted-toc

diff --git a/tests/fate/jxl.mak b/tests/fate/jxl.mak
index d653eb644a..1416b98f6d 100644
--- a/tests/fate/jxl.mak
+++ b/tests/fate/jxl.mak
@@ -19,6 +19,9 @@ fate-jxl-anim-demux: $(FATE_JPEGXL_ANIM_DEMUX)
 FATE_JPEGXL_PARSE += fate-jxl-small-ext-box
 fate-jxl-small-ext-box: CMD = framecrc -i $(TARGET_SAMPLES)/jxl/l.jxl -c copy
 
+FATE_JPEGXL_PARSE += fate-jxl-multiframe-permuted-toc
+fate-jxl-multiframe-permuted-toc: CMD = framecrc -i $(TARGET_SAMPLES)/jxl/orange.jxl -c copy
+
 FATE_JPEGXL_PARSE += $(FATE_JPEGXL_PARSE-yes)
 FATE_SAMPLES_FFMPEG-$(call FRAMECRC, IMAGE_JPEGXL_PIPE, , JPEGXL_PARSER) += $(FATE_JPEGXL_PARSE)
 fate-jxl-parse: $(FATE_JPEGXL_PARSE)
diff --git a/tests/ref/fate/jxl-multiframe-permuted-toc b/tests/ref/fate/jxl-multiframe-permuted-toc
new file mode 100644
index 0000000000..94fc008143
--- /dev/null
+++ b/tests/ref/fate/jxl-multiframe-permuted-toc
@@ -0,0 +1,11 @@
+#tb 0: 1/25
+#media_type 0: video
+#codec_id 0: jpegxl
+#dimensions 0: 2400x2400
+#sar 0: 0/1
+0,          0,          0,        1,   197602, 0x6f8fc984
+0,          1,          1,        1,    52335, 0x30421469
+0,          2,          2,        1,    20784, 0xaecb1df0
+0,          3,          3,        1,    11204, 0x6c7b0759
+0,          4,          4,        1,     5653, 0xa2ac56da
+0,          5,          5,        1,    11204, 0x6c7b0759
-- 
2.43.0

_______________________________________________
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 0/1] fate/jpegxl: add multiframe permuted TOC image parser

[Thilo Borgmann via ffmpeg-devel]

Am 09.12.23 um 20:49 schrieb Leo Izen:
> This patch requires a sample that hasn't been uploaded yet. It can be found
> at [1] and it should be placed at jxl/orange.jxl once uploaded. It's 262k,
> which is not very large, but large enough that I did not want to attach it
> to this email.
> 
> [1]: https://buzo.us/O.jxl
> 
> sample sha256sum: 9288337dc0d37effd1f539d4e20083a8ffed757e486f4098121520c74e8de6f6
> sample signature: https://buzo.us/A.asc

Uploaded.

-Thilo

_______________________________________________
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 0/1] fate/jpegxl: add multiframe permuted TOC image parser

[Andreas Rheinhardt]

Leo Izen:
> This patch requires a sample that hasn't been uploaded yet. It can be found
> at [1] and it should be placed at jxl/orange.jxl once uploaded. It's 262k,
> which is not very large, but large enough that I did not want to attach it
> to this email.
> 
> [1]: https://buzo.us/O.jxl
> 
> sample sha256sum: 9288337dc0d37effd1f539d4e20083a8ffed757e486f4098121520c74e8de6f6
> sample signature: https://buzo.us/A.asc
> 
> Leo Izen (1):
>   fate/jpegxl: add multiframe permuted TOC image parser test
> 
>  tests/fate/jxl.mak                         |  3 +++
>  tests/ref/fate/jxl-multiframe-permuted-toc | 11 +++++++++++
>  2 files changed, 14 insertions(+)
>  create mode 100644 tests/ref/fate/jxl-multiframe-permuted-toc
> 

I disagree with your assertion that 262k is not very large. Why do you
use a 2400x2400 sample?

- Andreas

_______________________________________________
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 0/1] fate/jpegxl: add multiframe permuted TOC image parser

[Leo Izen]

On 12/10/23 09:07, Andreas Rheinhardt wrote:
> Leo Izen:
>> This patch requires a sample that hasn't been uploaded yet. It can be found
>> at [1] and it should be placed at jxl/orange.jxl once uploaded. It's 262k,
>> which is not very large, but large enough that I did not want to attach it
>> to this email.
>>
>> [1]: https://buzo.us/O.jxl
>>
>> sample sha256sum: 9288337dc0d37effd1f539d4e20083a8ffed757e486f4098121520c74e8de6f6
>> sample signature: https://buzo.us/A.asc
>>
>> Leo Izen (1):
>>    fate/jpegxl: add multiframe permuted TOC image parser test
>>
>>   tests/fate/jxl.mak                         |  3 +++
>>   tests/ref/fate/jxl-multiframe-permuted-toc | 11 +++++++++++
>>   2 files changed, 14 insertions(+)
>>   create mode 100644 tests/ref/fate/jxl-multiframe-permuted-toc
>>
> 
> I disagree with your assertion that 262k is not very large. Why do you
> use a 2400x2400 sample?
> 

It had to be larger than 2048x2048 to have multiple LF groups in a 
single frame, as that's the LF group size. The images themselves are 
monochrome.

- Leo Izen (Traneptora)
_______________________________________________
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".