Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: use LUT for 9 and 18 codebook decompanding

2020-08-03 Thread Paul B Mahol
On 8/3/20, Martin Storsjö wrote: > On Mon, 3 Aug 2020, Paul B Mahol wrote: > >> ffmpeg | branch: master | Paul B Mahol | Mon Aug 3 >> 09:33:36 2020 +0200| [05e58ce4e29ea47b2e06888c64055aa2f8d3e76c] | >> committer: Paul B Mahol >> >> avcodec/cfhd: use LUT for 9 and 18 codebook decompanding >> >>

Re: [FFmpeg-devel] [PATCH V3 1/2] dnn/native: add native support for avg_pool

2020-08-03 Thread Fu, Ting
> -Original Message- > From: ffmpeg-devel On Behalf Of Guo, > Yejun > Sent: Monday, August 3, 2020 10:20 AM > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH V3 1/2] dnn/native: add native support for > avg_pool > > > > > -Original Message-

Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: Check CRCs for layer1 and layer2

2020-08-03 Thread Lynne
Aug 3, 2020, 18:31 by mich...@niedermayer.cc: > This differs from the MPEG specification as the actual real world > files do compute their CRC over variable areas and not the fixed > ones listed in the specification. This is also the reason for > the complexity of this code and the need to perform

Re: [FFmpeg-devel] [PATCH 1/2] lavf/url: add ff_url_decompose().

2020-08-03 Thread Marton Balint
On Thu, 30 Jul 2020, Nicolas George wrote: Signed-off-by: Nicolas George --- libavformat/tests/url.c | 34 +++ libavformat/url.c | 74 + libavformat/url.h | 41 +++ tests/ref/fate/url | 45 ++

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2020 at 02:07:36PM +0100, Jack Haughton wrote: > A NULL check in av_parse_video_rate() would certainly not be a bad idea. It > wouldn't (on its own) restore the NULL input robustness to > set_string_video_rate() though, as any error value returned from > av_parse_video_rate() would

Re: [FFmpeg-devel] [PATCH 0/7 v3] 22.2 channel layout support for AAC decoding

2020-08-03 Thread Jan Ekström
On Sat, Aug 1, 2020 at 2:07 PM Jan Ekström wrote: > > The decoding parts of this patch set have not changed from v2. > I have now added changes so that 22.2 gets properly handled as > 5.1 (for which it has backwards compatibility) until it gets a > proper remix matrix defined in swresample. So >

Re: [FFmpeg-devel] [PATCH 1/2] avformat/dv: allow returning damaged audio

2020-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2020 at 10:38:21PM +0200, Marton Balint wrote: > > > On Sun, 2 Aug 2020, Dave Rice wrote: > > > > > > > > On Aug 1, 2020, at 5:26 PM, Marton Balint wrote: > > > > > > > > > > > > On Sat, 1 Aug 2020, Michael Niedermayer wrote: > > > > > > > On Sat, Aug 01, 2020 at 07:28:53P

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/cfhd: use LUT for 9 and 18 codebook decompanding

2020-08-03 Thread Martin Storsjö
On Mon, 3 Aug 2020, Paul B Mahol wrote: ffmpeg | branch: master | Paul B Mahol | Mon Aug 3 09:33:36 2020 +0200| [05e58ce4e29ea47b2e06888c64055aa2f8d3e76c] | committer: Paul B Mahol avcodec/cfhd: use LUT for 9 and 18 codebook decompanding Also fix codebook 9 decompanding, fixing artifact wit

Re: [FFmpeg-devel] [PATCH 3/3] test: hlsenc: Use unique init/segment file names for the fmp4_ac3 test

2020-08-03 Thread Martin Storsjö
On Sun, 2 Aug 2020, Martin Storsjö wrote: On Sunday, August 2, 2020, Andreas Rheinhardt wrote: Martin Storsjö: > Previously, the hls-fmp4 and hls-fmp4_ac3 tests used the same file > names for init and segment files, which occasionally could cause > corruption

Re: [FFmpeg-devel] [PATCH 1/2] test: hlsenc: Don't generate test data with -re

2020-08-03 Thread Martin Storsjö
On Mon, 3 Aug 2020, Michael Niedermayer wrote: On Sun, Aug 02, 2020 at 11:38:51PM +0300, Martin Storsjö wrote: This parameter artificially throttled the generation of the test sample to take 5 seconds. --- tests/fate/hlsenc.mak | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH 1/2] avformat/dv: allow returning damaged audio

2020-08-03 Thread Marton Balint
On Sun, 2 Aug 2020, Dave Rice wrote: On Aug 1, 2020, at 5:26 PM, Marton Balint wrote: On Sat, 1 Aug 2020, Michael Niedermayer wrote: On Sat, Aug 01, 2020 at 07:28:53PM +0200, Marton Balint wrote: On Sat, 1 Aug 2020, Michael Niedermayer wrote: Fixes: Ticket8762 Signed-off-by: Mich

Re: [FFmpeg-devel] [PATCH 1/2] test: hlsenc: Don't generate test data with -re

2020-08-03 Thread Michael Niedermayer
On Sun, Aug 02, 2020 at 11:38:51PM +0300, Martin Storsjö wrote: > This parameter artificially throttled the generation of the test sample > to take 5 seconds. > --- > tests/fate/hlsenc.mak | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/fate/hlsenc.mak b/tests/fate/

Re: [FFmpeg-devel] [PATCH v2] [RFC] lavc, lavfmt: add FLIF decoding support

2020-08-03 Thread Kartik K. Khullar
Wow, That is such an awesome idea. Did not know about that offsetof() thing earlier. Definitely going to use this idea. > IIRC, the Linux kernel makes heavy use of this structure. > That's really cool. Actually I implemented linked lists and not dynamic arrays because some elements from somewhe

Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Lynne
Aug 3, 2020, 15:43 by jamr...@gmail.com: > Layers 1 and 2 use lengths in bits which are not a multiple of 8, > and our CRC works on a per-byte basis. > > Based on b48397e7b8 > LGTM, that was a really bad messup on my part. Thanks. ___ ffmpeg-devel maili

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: use LUT for 9 and 18 codebook decompanding

2020-08-03 Thread Kieran Kunhya
On Mon, 3 Aug 2020 at 10:48, Paul B Mahol wrote: > Hi, > > patch attached. > > Seems fine if tested. Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or ema

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-03 Thread Jack Haughton
A NULL check in av_parse_video_rate() would certainly not be a bad idea. It wouldn't (on its own) restore the NULL input robustness to set_string_video_rate() though, as any error value returned from av_parse_video_rate() would result in val being logged using %s, which is the whole problem that a5

Re: [FFmpeg-devel] Inconsistent handling of initial_padding and PTS adjustment

2020-08-03 Thread Anton Khirnov
Quoting Kieran Kunhya (2020-08-03 18:36:16) > On Mon, 3 Aug 2020 at 08:16, Anton Khirnov wrote: > > > Quoting Kieran Kunhya (2020-07-27 20:15:17) > > > On Mon, 27 Jul 2020 at 11:09, Anton Khirnov wrote: > > > > > > > Quoting Kieran Kunhya (2020-07-26 01:51:22) > > > > > Hi, > > > > > > > > > > I

Re: [FFmpeg-devel] [PATCH] doc/developer: origin of tables should be documented.

2020-08-03 Thread Zhao Zhili
> On Aug 3, 2020, at 5:54 PM, Nicolas George wrote: > > Nicolas George (12020-07-31): >> Something like this would be acceptable: >> >> /* Reverse-engineered by encoding various files with the reference >> encoder. */ >> >> Reluctance to give this little information is baffling to me. > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2020 at 10:47:44AM +0200, Lynne wrote: > Aug 3, 2020, 01:45 by mich...@niedermayer.cc: > > > On Sun, Aug 02, 2020 at 08:52:22PM +, Lynne wrote: > > > >> ffmpeg | branch: master | Lynne | Sun Aug 2 22:45:00 2020 > >> +0200| [b48397e7b84864f2d4c70361a4c4bed93e826753] | committ

[FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: Check CRCs for layer1 and layer2

2020-08-03 Thread Michael Niedermayer
This differs from the MPEG specification as the actual real world files do compute their CRC over variable areas and not the fixed ones listed in the specification. This is also the reason for the complexity of this code and the need to perform the CRC check for layer2 in the middle of layer2 decod

Re: [FFmpeg-devel] Inconsistent handling of initial_padding and PTS adjustment

2020-08-03 Thread Kieran Kunhya
On Mon, 3 Aug 2020 at 08:16, Anton Khirnov wrote: > Quoting Kieran Kunhya (2020-07-27 20:15:17) > > On Mon, 27 Jul 2020 at 11:09, Anton Khirnov wrote: > > > > > Quoting Kieran Kunhya (2020-07-26 01:51:22) > > > > Hi, > > > > > > > > I notice that some encoders adjust the PTS with initial_padding

Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2020 at 10:43:11AM -0300, James Almer wrote: > Layers 1 and 2 use lengths in bits which are not a multiple of 8, > and our CRC works on a per-byte basis. > > Based on b48397e7b8 > > Signed-off-by: James Almer > --- > libavcodec/mpegaudiodec_template.c | 2 +- > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] doc/developer: origin of tables should be documented.

2020-08-03 Thread Valentin Schweitzer
Can other developers give their input on this? I am not currently a FFmpeg developer, so my opinion might be less relevant than what others have stated. I think giving a source for things that are not inherently obvious, but are part of the code is a good idea. Even if some tables are well ex

Re: [FFmpeg-devel] Create and populate AVStream side data packet with contents of ISOBMFF edit list entries

2020-08-03 Thread Derek Buitenhuis
On 30/07/2020 16:55, Matthew Szatmary wrote: >> This seems to be really MP4 centric. Did you check if such patch sent out >> for RFC in 2018 could do it? >> >> https://patchwork.ffmpeg.org/project/ffmpeg/patch/1522179841-34881-2-git-send-email-derek.buitenh...@gmail.com/ >> > I was unaware of this

Re: [FFmpeg-devel] [PATCH v2] [RFC] lavc, lavfmt: add FLIF decoding support

2020-08-03 Thread Nicolas George
Kartik K. Khullar (12020-08-03): > > Mixing linked lists and indices is usually a sign you are doing > > something wrong. > > > > Here, I believe you would do better with a doubly linked list and > > pointers instead of indices. > > > > Note: there is a trick with doubly linked lists where you put

Re: [FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread James Almer
On 8/3/2020 10:45 AM, Lynne wrote: > Aug 3, 2020, 15:43 by jamr...@gmail.com: > >> Layers 1 and 2 use lengths in bits which are not a multiple of 8, >> and our CRC works on a per-byte basis. >> >> Based on b48397e7b8 >> > > LGTM, that was a really bad messup on my part. > Thanks. Will apply. Tha

Re: [FFmpeg-devel] [PATCH] fate: Update the test references for h264-skip-nokey and h264-skip-noparse after b48397e7b84864f2

2020-08-03 Thread James Almer
On 8/3/2020 10:26 AM, James Almer wrote: > On 8/3/2020 9:45 AM, Michael Niedermayer wrote: >> On Mon, Aug 03, 2020 at 09:54:41AM +0300, Martin Storsjö wrote: >>> --- >>> tests/ref/fate/h264-skip-nointra | 86 >>> tests/ref/fate/h264-skip-nokey | 86 ++

[FFmpeg-devel] [PATCH] avcodec/mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread James Almer
Layers 1 and 2 use lengths in bits which are not a multiple of 8, and our CRC works on a per-byte basis. Based on b48397e7b8 Signed-off-by: James Almer --- libavcodec/mpegaudiodec_template.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mpegaudiodec_template.c

Re: [FFmpeg-devel] [PATCH] fate: Update the test references for h264-skip-nokey and h264-skip-noparse after b48397e7b84864f2

2020-08-03 Thread James Almer
On 8/3/2020 9:45 AM, Michael Niedermayer wrote: > On Mon, Aug 03, 2020 at 09:54:41AM +0300, Martin Storsjö wrote: >> --- >> tests/ref/fate/h264-skip-nointra | 86 >> tests/ref/fate/h264-skip-nokey | 86 >> 2 files changed, 86 inse

Re: [FFmpeg-devel] [PATCH] fate/aac: add missing bitexact flag to some encoder tests

2020-08-03 Thread James Almer
On 8/3/2020 7:25 AM, Paul B Mahol wrote: > On 8/3/20, James Almer wrote: >> On 8/2/2020 8:43 PM, James Almer wrote: >>> Will prevet FATE from breaking once LIBAVCODEC_VERSION_MINOR is bumped to >>> 100. >>> >>> Reported-by: zane >>> Signed-off-by: James Almer >>> --- >>> If anyone wants to check

Re: [FFmpeg-devel] [FFmpeg-cvslog] mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Lynne
Aug 3, 2020, 01:45 by mich...@niedermayer.cc: > On Sun, Aug 02, 2020 at 08:52:22PM +, Lynne wrote: > >> ffmpeg | branch: master | Lynne | Sun Aug 2 22:45:00 2020 >> +0200| [b48397e7b84864f2d4c70361a4c4bed93e826753] | committer: Lynne >> >> mpegaudiodec_template: disable CRC checking for lay

Re: [FFmpeg-devel] [PATCH] fate: Update the test references for h264-skip-nokey and h264-skip-noparse after b48397e7b84864f2

2020-08-03 Thread Michael Niedermayer
On Mon, Aug 03, 2020 at 09:54:41AM +0300, Martin Storsjö wrote: > --- > tests/ref/fate/h264-skip-nointra | 86 > tests/ref/fate/h264-skip-nokey | 86 > 2 files changed, 86 insertions(+), 86 deletions(-) b48397e7b84864f2d4c70361a4

Re: [FFmpeg-devel] [FFmpeg-cvslog] mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Lynne
Aug 3, 2020, 08:58 by mar...@martin.st: > On Sun, 2 Aug 2020, Lynne wrote: > >> ffmpeg | branch: master | Lynne | Sun Aug 2 22:45:00 2020 >> +0200| [b48397e7b84864f2d4c70361a4c4bed93e826753] | committer: Lynne >> >> mpegaudiodec_template: disable CRC checking for layers 1 and 2 >> >> Layers 1 a

Re: [FFmpeg-devel] [PATCH] fate: Update the test references for h264-skip-nokey and h264-skip-noparse after b48397e7b84864f2

2020-08-03 Thread myp...@gmail.com
On Mon, Aug 3, 2020 at 8:00 PM myp...@gmail.com wrote: > > On Mon, Aug 3, 2020 at 2:54 PM Martin Storsjö wrote: > > > > --- > > tests/ref/fate/h264-skip-nointra | 86 > > tests/ref/fate/h264-skip-nokey | 86 > > 2 files changed,

Re: [FFmpeg-devel] [PATCH] fate: Update the test references for h264-skip-nokey and h264-skip-noparse after b48397e7b84864f2

2020-08-03 Thread myp...@gmail.com
On Mon, Aug 3, 2020 at 2:54 PM Martin Storsjö wrote: > > --- > tests/ref/fate/h264-skip-nointra | 86 > tests/ref/fate/h264-skip-nokey | 86 > 2 files changed, 86 insertions(+), 86 deletions(-) > > diff --git a/tests/ref/fate/h26

Re: [FFmpeg-devel] [PATCH] [RFC] libavcodec/hevc_refs: Clear DPB of old frames

2020-08-03 Thread Kieran Kunhya
On Mon, 3 Aug 2020 at 09:18, Anton Khirnov wrote: > Quoting Kieran Kunhya (2020-07-16 21:32:34) > > During glitching or looping streams, old frames remain in the DPB. > > The decoder incorrectly thinks that the DPB contains the right number of > > buffered frames to output and reordering breaks b

[FFmpeg-devel] [v5] dnn_backend_native_layer_mathunary: add ceil support

2020-08-03 Thread Mingyu Yin
It can be tested with the model generated with below python script: import tensorflow as tf import os import numpy as np import imageio from tensorflow.python.framework import graph_util name = 'ceil' pb_file_path = os.getcwd() if not os.path.exists(pb_file_path+'/{}_savemodel/'.format(name)):

Re: [FFmpeg-devel] [PATCH] fate/aac: add missing bitexact flag to some encoder tests

2020-08-03 Thread Paul B Mahol
On 8/3/20, James Almer wrote: > On 8/2/2020 8:43 PM, James Almer wrote: >> Will prevet FATE from breaking once LIBAVCODEC_VERSION_MINOR is bumped to >> 100. >> >> Reported-by: zane >> Signed-off-by: James Almer >> --- >> If anyone wants to check why adding LIBAVCODEC_IDENT to the bitstream (or >>

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Jan Ekström
On Mon, Aug 3, 2020 at 10:34 AM Carl Eugen Hoyos wrote: > > > > > Am 02.08.2020 um 23:16 schrieb Jan Ekström : > > > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > >> > >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > >>> > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen

Re: [FFmpeg-devel] [PATCH] doc/developer: origin of tables should be documented.

2020-08-03 Thread Nicolas George
Nicolas George (12020-07-31): > Something like this would be acceptable: > > /* Reverse-engineered by encoding various files with the reference >encoder. */ > > Reluctance to give this little information is baffling to me. Can other developers give their input on this? Is it acceptable or t

[FFmpeg-devel] [PATCH] avcodec/cfhd: use LUT for 9 and 18 codebook decompanding

2020-08-03 Thread Paul B Mahol
Hi, patch attached. 0001-avcodec-cfhd-use-LUT-for-9-and-18-codebook-decompand.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or em

[FFmpeg-devel] [PATCH] lavu/buffer: add a convenience function for replacing buffers

2020-08-03 Thread Anton Khirnov
A common pattern e.g. in libavcodec is replacing/updating buffer references: unref old one, ref new one. This function allows simplifying such code an avoiding unnecessary refs+unrefs if the references are already equivalent. --- Updated to not touch dst on failure. Still not using buffer_replace(

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Hendrik Leppkes
On Mon, Aug 3, 2020 at 9:34 AM Carl Eugen Hoyos wrote: > > > > > Am 02.08.2020 um 23:16 schrieb Jan Ekström : > > > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: > >> > >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : > >>> > On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen H

Re: [FFmpeg-devel] [FFmpeg-cvslog] mpegaudiodec_template: disable CRC checking for layers 1 and 2

2020-08-03 Thread Martin Storsjö
On Mon, 3 Aug 2020, Lynne wrote: Aug 3, 2020, 08:58 by mar...@martin.st: This change broke two fate tests; fate-h264-skip-nokey and fate-h264-skip-nointra. The change does look sensible in itself though (framecrcs that used to be all zeros now have nonzero, varying values). Someone must h

Re: [FFmpeg-devel] [PATCH] [RFC] libavcodec/hevc_refs: Clear DPB of old frames

2020-08-03 Thread Anton Khirnov
Quoting Kieran Kunhya (2020-07-16 21:32:34) > During glitching or looping streams, old frames remain in the DPB. > The decoder incorrectly thinks that the DPB contains the right number of > buffered frames to output and reordering breaks badly > > Any non-cosmetic comments welcome. >From 23f027209

Re: [FFmpeg-devel] [PATCH 1/5] avutil/channel_layout: add 22.2 layout

2020-08-03 Thread Carl Eugen Hoyos
> Am 02.08.2020 um 23:16 schrieb Jan Ekström : > >> On Sat, Aug 1, 2020 at 4:29 PM Carl Eugen Hoyos wrote: >> >>> Am Sa., 1. Aug. 2020 um 14:47 Uhr schrieb Jan Ekström : >>> On Sat, Aug 1, 2020 at 3:08 PM Carl Eugen Hoyos wrote: > Am Sa., 1. Aug. 2020 um 14:06 Uhr schrieb Jan

Re: [FFmpeg-devel] Inconsistent handling of initial_padding and PTS adjustment

2020-08-03 Thread Anton Khirnov
Quoting Kieran Kunhya (2020-07-27 20:15:17) > On Mon, 27 Jul 2020 at 11:09, Anton Khirnov wrote: > > > Quoting Kieran Kunhya (2020-07-26 01:51:22) > > > Hi, > > > > > > I notice that some encoders adjust the PTS with initial_padding and some > > > don't. > > > Is this intentional and should we de