Re: [FFmpeg-devel] [PATCH 08/10] libavutil: add side data AVDnnBoundingBox for dnn based detect/classify filters

2021-02-10 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: 2021年2月10日 12:59 > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH 08/10] libavutil: add side data AVDnnBoundingBox for dnn > based detect/classify filters > > Signed-off-by: Guo, Yejun > --- > doc/APIchanges | 2 +

[FFmpeg-devel] [PATCH V2 02/10] dnn_backend_openvino.c: fix multi-thread issue for async execution

2021-02-10 Thread Guo, Yejun
once we mark done for the task in function infer_completion_callback, the task is possible to be release in function ff_dnn_get_async_result_ov in another thread just after it, so we need to record request queue first, instead of using task->ov_model->request_queue later. Signed-off-by: Guo, Yejun

[FFmpeg-devel] [PATCH V2 01/10] dnn_backend_openvino.c: fix mismatch between ffmpeg(NHWC) and openvino(NCHW)

2021-02-10 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- libavfilter/dnn/dnn_backend_openvino.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c index beca256390..48f5ba50be 100644 --- a/libavfilter/dn

[FFmpeg-devel] [PATCH V2 03/10] dnn: extract common functions used by different filters

2021-02-10 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- libavfilter/Makefile| 1 + libavfilter/dnn_filter_common.c | 106 libavfilter/dnn_filter_common.h | 59 ++ libavfilter/vf_derain.c | 46 +++--- libavfilter/vf_dnn_processing.c | 93

[FFmpeg-devel] [PATCH V2 04/10] dnn: add function type for model

2021-02-10 Thread Guo, Yejun
So the backend knows the usage of model is for frame processing, detect, classify, etc. Each function type has different behavior in backend when handling the input/output data of the model. Signed-off-by: Guo, Yejun --- libavfilter/dnn/dnn_backend_native.c | 3 ++- libavfilter/dnn/dnn_backen

[FFmpeg-devel] [PATCH V2 05/10] dnn_interface.h: add enum DNNColorOrder

2021-02-10 Thread Guo, Yejun
the data type and order together decide the color format, we could not use AVPixelFormat directly because not all the possible formats are covered by it. Signed-off-by: Guo, Yejun --- libavfilter/dnn_interface.h | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavfil

[FFmpeg-devel] [PATCH V2 06/10] dnn: add color conversion for analytic case

2021-02-10 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- libavfilter/dnn/dnn_backend_native.c | 2 +- libavfilter/dnn/dnn_backend_openvino.c | 23 ++- libavfilter/dnn/dnn_backend_tf.c | 2 +- libavfilter/dnn/dnn_io_proc.c | 56 +- libavfilter/dnn/dnn_io_proc.h |

[FFmpeg-devel] [PATCH V2 09/10] libavfilter: show side data of dnn bounding box

2021-02-10 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- libavfilter/f_sidedata.c | 2 ++ libavfilter/vf_showinfo.c | 34 ++ 2 files changed, 36 insertions(+) diff --git a/libavfilter/f_sidedata.c b/libavfilter/f_sidedata.c index 3757723375..08ee96c4ef 100644 --- a/libavfilter/f_sidedata.

[FFmpeg-devel] [PATCH V2 10/10] libavfilter: add filter dnn_detect for object detection

2021-02-10 Thread Guo, Yejun
Below are the example steps to do object detection: 1. download and install l_openvino_toolkit_p_2021.1.110.tgz from https://software.intel.com/content/www/us/en/develop/tools/openvino-toolkit/download.html or, We can get source code (tag 2021.1), build and install. 2. export LD_LIBRARY_PATH wit

[FFmpeg-devel] [PATCH V2 07/10] dnn_backend_openvino.c: allow out_frame as NULL for analytic case

2021-02-10 Thread Guo, Yejun
From: Ting Fu --- libavfilter/dnn/dnn_backend_openvino.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c index cca155a52c..5be053b7f8 100644 --- a/libavfilter/dnn/dnn_backend_openvino.c +++ b

[FFmpeg-devel] [PATCH V2 08/10] libavutil: add side data AVDnnBoundingBox for dnn based detect/classify filters

2021-02-10 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- doc/APIchanges | 2 ++ libavutil/Makefile | 1 + libavutil/dnn_bbox.h | 68 libavutil/frame.c| 1 + libavutil/frame.h| 7 + libavutil/version.h | 2 +- 6 files changed, 80 insertions(+), 1 deletio

Re: [FFmpeg-devel] [PATCH] avfilter: add aexciter audio filter

2021-02-10 Thread Paul B Mahol
Will apply soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add exposure video filter

2021-02-10 Thread Paul B Mahol
Will apply soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add lp option

2021-02-10 Thread Anton Khirnov
Quoting Christopher Degawa (2021-02-09 19:58:16) > From: Christopher Degawa > > Equivalent to the --lp option for SvtAv1EncApp, and is the only way > to control how much cpu power svt-av1 uses for now > > Not using thread_count as it would be preferable to reserve that until > svt-av1 properly i

Re: [FFmpeg-devel] [PATCH] avdevice/avdevice: Deprecate AVDevice Capabilities API

2021-02-10 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2021-02-09 09:04:23) > Can I get an update on how to proceed with this patch? It doesn't seem that anyone actually objected to this patch, so go ahead and push IMO. > > - Andreas > > PS: I could already remove all the av_device_capabilities options > (except the sent

Re: [FFmpeg-devel] [PATCH v2] Add support for the new key & value API in libaom.

2021-02-10 Thread Jan Ekström
On Wed, Feb 10, 2021 at 9:16 AM Bohan Li wrote: > > Hi Jan, > > Yes the modified patch looks good to me. > Please let me know if there is anything needed from my end. > > Thank you very much! > > Bohan > Cheers, pulled in as 82aab8a4eec33ee92c92c7679a4d7e6f03b109b4 . Jan

Re: [FFmpeg-devel] [PATCH] avformat/movenc: remove unnecessary HDR metadata log messages

2021-02-10 Thread Anton Khirnov
Quoting Jan Ekström (2021-02-10 00:31:51) > These made sense before 3ebf449766d4b69a7774c4990fb24fe71c848a3e > when the creation of these boxes was specifically requested by the > user, but now they have become unnecessary as they are just like > many other boxes: If the input has the information r

Re: [FFmpeg-devel] [PATCH] avformat/movenc: remove unnecessary HDR metadata log messages

2021-02-10 Thread Paul B Mahol
lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] [RFC][v3] Tech Resolution Process

2021-02-10 Thread Anton Khirnov
Quoting Lynne (2021-02-08 20:13:39) > Feb 8, 2021, 15:23 by j...@videolan.org: > > + > > +The decisions from the TC are final, until the matters are reopened after > > +no less than one year, by either the GA or the TC auto-seizing. > > > > Is there an OR there? Can the question be raised again af

Re: [FFmpeg-devel] [PATCH] avdevice/avdevice: Deprecate AVDevice Capabilities API

2021-02-10 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2021-02-09 09:04:23) >> Can I get an update on how to proceed with this patch? > > It doesn't seem that anyone actually objected to this patch, so go ahead > and push IMO. > >> >> - Andreas >> >> PS: I could already remove all the av_device_capabilitie

Re: [FFmpeg-devel] [PATCH] [RFC][v3] Tech Resolution Process

2021-02-10 Thread Lynne
Feb 10, 2021, 11:45 by an...@khirnov.net: > Quoting Lynne (2021-02-08 20:13:39) > >> Feb 8, 2021, 15:23 by j...@videolan.org: >> > + >> > +The decisions from the TC are final, until the matters are reopened after >> > +no less than one year, by either the GA or the TC auto-seizing. >> > >> >> Is t

Re: [FFmpeg-devel] [PATCH] [RFC][v3] Tech Resolution Process

2021-02-10 Thread Anton Khirnov
Quoting Lynne (2021-02-10 14:15:29) > Feb 10, 2021, 11:45 by an...@khirnov.net: > > > Quoting Lynne (2021-02-08 20:13:39) > > > >> Feb 8, 2021, 15:23 by j...@videolan.org: > >> > + > >> > +The decisions from the TC are final, until the matters are reopened > >> > after > >> > +no less than one ye

Re: [FFmpeg-devel] [PATCH] [RFC][v3] Tech Resolution Process

2021-02-10 Thread Lynne
Feb 10, 2021, 14:34 by an...@khirnov.net: > Quoting Lynne (2021-02-10 14:15:29) > >> Feb 10, 2021, 11:45 by an...@khirnov.net: >> >> > Quoting Lynne (2021-02-08 20:13:39) >> > >> >> Feb 8, 2021, 15:23 by j...@videolan.org: >> >> > + >> >> > +The decisions from the TC are final, until the matters a

Re: [FFmpeg-devel] [PATCH] x11grab: capture a window instead of the whole screen

2021-02-10 Thread sgerwk-at-aol . com
On Sat, 6 Feb 2021, Paul B Mahol wrote: What happens if you use non-existing window? I added an error message for this case, but the output video is still created (empty). Is there a value I can return to avoid this? On Sun, 7 Feb 2021, Andriy Gelman wrote: Also the commit title should star

[FFmpeg-devel] [PATCH] lavu/tx: support in-place FFT transforms

2021-02-10 Thread Lynne
This commit adds support for in-place FFT transforms. Since our internal transforms were all in-place anyway, this only changes the permutation on the input. Unfortunately, research papers were of no help here. All focused on dry hardware implementations, where permutes are free, or on software i

[FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Christopher Degawa
From: Christopher Degawa Equivalent to the --lp option for SvtAv1EncApp, and is the only way to control how much cpu power svt-av1 uses for now Not using thread_count as it would be preferable to reserve that until svt-av1 properly implements a threads option 0 == getconf _NPROCESSORS_ONLN Sig

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add lp option

2021-02-10 Thread Christopher Degawa
> > It is very non-obvious what "lp" means, "logical_processors" would be > better IMO. > Okay, I will change it to `-logical_processors` ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscri

[FFmpeg-devel] [PATCH] avcodec/hevcdec: check that the local context list exists before dereferencing it

2021-02-10 Thread James Almer
Since the decoder is not flagged as init cleanup capable, hevc_decode_free() is being called manually if the hevc_decode_extradata() call fails at the end of hevc_decode_init(). In a frame threading scenario, however, if AVCodec->init() returns an error, ff_frame_thread_free() will be called regard

Re: [FFmpeg-devel] [PATCH 3/8] avformat/mov: simplify size code in probing a bit

2021-02-10 Thread Michael Niedermayer
On Sat, Feb 06, 2021 at 11:06:47PM +0100, Paul B Mahol wrote: > LGTM will apply thx {...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No great genius has ever existed without some touch of madness. -- Aristotle signature.asc Description: PGP signature _

Re: [FFmpeg-devel] [PATCH] avcodec/hevcdec: check that the local context list exists before dereferencing it

2021-02-10 Thread Andreas Rheinhardt
James Almer: > Since the decoder is not flagged as init cleanup capable, hevc_decode_free() > is being called manually if the hevc_decode_extradata() call fails at the end > of hevc_decode_init(). > In a frame threading scenario, however, if AVCodec->init() returns an error, > ff_frame_thread_free(

Re: [FFmpeg-devel] [PATCH 2/8] avformat/mov: Support size = 1 and size = 0 special cases in probing

2021-02-10 Thread Michael Niedermayer
On Mon, Feb 08, 2021 at 02:25:50PM +0100, Michael Niedermayer wrote: > On Sat, Feb 06, 2021 at 11:33:38AM -0800, Chad Fraleigh wrote: > > On 2/6/2021 9:22 AM, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > > --- > > > libavformat/mov.c | 5 + > > > 1 file changed, 5

Re: [FFmpeg-devel] [PATCH 4/8] avformat/mov: Factor offset advancement out in probing

2021-02-10 Thread Michael Niedermayer
On Sat, Feb 06, 2021 at 11:07:35PM +0100, Paul B Mahol wrote: > LGTM will apply {...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Opposition brings concord. Out of discord comes the fairest harmony. -- Heraclitus signature.asc Description: PGP signature

Re: [FFmpeg-devel] [PATCH v3] avformat/concatdec: add support for setting input options

2021-02-10 Thread Nicolas George
Jan Ekström (12021-02-08): > This way protocol or format related options can be set for all > of the files opened during concatenation both globally as well > as per-file. > --- > > Changes from v2: > > 1. Added an example, although I had issues figuring out something useful >that is not a ha

Re: [FFmpeg-devel] [PATCH]lavfi/vflip: Support Bayer vertical flip

2021-02-10 Thread Paul B Mahol
Just apply this, it is very unlikely that there are faster ways. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subjec

Re: [FFmpeg-devel] [PATCH v3] avformat/concatdec: add support for setting input options

2021-02-10 Thread Jan Ekström
On Wed, Feb 10, 2021 at 8:33 PM Nicolas George wrote: > > Jan Ekström (12021-02-08): > > This way protocol or format related options can be set for all > > of the files opened during concatenation both globally as well > > as per-file. > > --- > > > > Changes from v2: > > > > 1. Added an example,

Re: [FFmpeg-devel] [PATCH] avformat/movenc: remove unnecessary HDR metadata log messages

2021-02-10 Thread Jan Ekström
On Wed, Feb 10, 2021 at 12:43 PM Paul B Mahol wrote: > > lgtm Cheers, applied as 752f14ff942576dd8697320fb6ba055593a2de9d . Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visi

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flvdec: Treat high ts byte as unsigned

2021-02-10 Thread Michael Niedermayer
On Tue, Nov 24, 2020 at 12:42:02AM +0100, Michael Niedermayer wrote: > Fixes: left shift of 255 by 24 places cannot be represented in type 'int' > Fixes: > 27516/clusterfuzz-testcase-minimized-ffmpeg_dem_KUX_fuzzer-5152854660349952 > > Signed-off-by: Michael Niedermayer > --- > libavformat/flvd

Re: [FFmpeg-devel] [PATCH] lavu/tx: support in-place FFT transforms

2021-02-10 Thread Lynne
Feb 10, 2021, 18:15 by d...@lynne.ee: > This commit adds support for in-place FFT transforms. Since our > internal transforms were all in-place anyway, this only changes > the permutation on the input. > > Unfortunately, research papers were of no help here. All focused > on dry hardware implemen

Re: [FFmpeg-devel] [PATCH] libavcodec/hevcdsp: port SIMD idct functions from 32-bit.

2021-02-10 Thread Martin Storsjö
Hi Reimar, On Sat, 16 Jan 2021, Martin Storsjö wrote: +.macro idct_16x16 bitdepth +function ff_hevc_idct_16x16_\bitdepth\()_neon, export=1 +//r0 - coeffs +mov x15, lr + Binutils doesn't recognize "lr" as alias for x30 It didn’t have an issue in the Debian unstable VM? Th

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Mark Thompson
On 10/02/2021 17:16, Christopher Degawa wrote: From: Christopher Degawa Equivalent to the --lp option for SvtAv1EncApp, and is the only way to control how much cpu power svt-av1 uses for now Not using thread_count as it would be preferable to reserve that until svt-av1 properly implements a th

Re: [FFmpeg-devel] [PATCH V2 08/10] libavutil: add side data AVDnnBoundingBox for dnn based detect/classify filters

2021-02-10 Thread Mark Thompson
On 10/02/2021 09:34, Guo, Yejun wrote: Signed-off-by: Guo, Yejun --- doc/APIchanges | 2 ++ libavutil/Makefile | 1 + libavutil/dnn_bbox.h | 68 libavutil/frame.c| 1 + libavutil/frame.h| 7 + libavutil/version.h | 2 +

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Christopher Degawa
Based on my limited understanding of the code, it's limiting the usage using pthread_setaffinity_np and CPU_SET on Linux to limit the process to certain CPUs, but it also has a default and max of the return of `sysconf(_SC_NPROCESSORS_ONLN)`. According to Hassene Tmar of SVT-AV1, it is a "target co

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-10 Thread Paul B Mahol
On Wed, Feb 10, 2021 at 11:08 PM Mark Thompson wrote: > On 10/02/2021 17:16, Christopher Degawa wrote: > > From: Christopher Degawa > > > > Equivalent to the --lp option for SvtAv1EncApp, and is the only way > > to control how much cpu power svt-av1 uses for now > > > > Not using thread_count as

Re: [FFmpeg-devel] [PATCH] avcodec/hevcdec: check that the local context list exists before dereferencing it

2021-02-10 Thread Paul B Mahol
On Wed, Feb 10, 2021 at 6:57 PM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > James Almer: > > Since the decoder is not flagged as init cleanup capable, > hevc_decode_free() > > is being called manually if the hevc_decode_extradata() call fails at > the end > > of hevc_decode_init().