Re: [FFmpeg-devel] [PATCH 4/4] avfilter/vf_v360: refactor (i)flat_range for fisheye

2021-04-18 Thread Paul B Mahol
On Mon, Apr 12, 2021 at 7:44 AM Daniel Playfair Cal < daniel.playfair@gmail.com> wrote: > > Ok, dfov for width != height in (d)fisheye have been fixed. > > Great, thanks :) > > > The aspect ratio one breaks handling in case input is in equirectangular > format. > > That's not obvious to me - c

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Gyan Doshi > Sent: 2021年4月18日 13:04 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for > unknown command > > > > On 2021-04-18 10:11, Guo, Yejun wrote: > > The build log: > >

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/nvenc: move lossless presets after new ones

2021-04-18 Thread Timo Rothenpieler
pushed a refactor of the entire logic, since I realized that another piece of code (the options parsing logic) relies on the new presets being last. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffm

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/af_mcompand: check allocation results

2021-04-18 Thread Marton Balint
On Sat, 10 Apr 2021, Marton Balint wrote: On Sat, 10 Apr 2021, Andreas Rheinhardt wrote: Marton Balint: Fixes the only remaining part of ticket #8931. Signed-off-by: Marton Balint --- libavfilter/af_mcompand.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavfilter/af_mcomp

[FFmpeg-devel] [PATCH 1/6] lavfi/dnn_backend_openvino.c: unify code for infer request for sync/async

2021-04-18 Thread Guo, Yejun
--- libavfilter/dnn/dnn_backend_openvino.c | 49 +++--- 1 file changed, 21 insertions(+), 28 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c index 0757727a9c..874354ecef 100644 --- a/libavfilter/dnn/dnn_backend_openvin

[FFmpeg-devel] [PATCH 2/6] lavfi/dnn_backend_openvino.c: add InferenceItem between TaskItem and RequestItem

2021-04-18 Thread Guo, Yejun
There's one task item for one function call from dnn interface, there's one request item for one call to openvino. For classify, one task might need multiple inference for classification on every bounding box, so add InferenceItem. --- libavfilter/dnn/dnn_backend_openvino.c | 157 +

[FFmpeg-devel] [PATCH 3/6] lavfi/dnn_backend_openvino.c: move the logic for batch mode earlier

2021-04-18 Thread Guo, Yejun
--- libavfilter/dnn/dnn_backend_openvino.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c index 3692a381e2..a695d863b5 100644 --- a/libavfilter/dnn/dnn_backend_openvino.c +++ b/libavf

[FFmpeg-devel] [PATCH 4/6] lavfi/dnn: refine dnn interface to add DNNExecBaseParams

2021-04-18 Thread Guo, Yejun
Different function type of model requires different parameters, for example, object detection detects lots of objects (cat/dog/...) in the frame, and classifcation needs to know which object (cat or dog) it is going to classify. The current interface needs to add a new function with more parameter

[FFmpeg-devel] [PATCH 5/6] lavfi/dnn: add classify support with openvino backend

2021-04-18 Thread Guo, Yejun
Signed-off-by: Guo, Yejun --- libavfilter/dnn/dnn_backend_openvino.c | 143 + libavfilter/dnn/dnn_io_proc.c | 60 +++ libavfilter/dnn/dnn_io_proc.h | 1 + libavfilter/dnn_filter_common.c| 21 libavfilter/dnn_filter_common.h

[FFmpeg-devel] [PATCH 6/6] lavfi/dnn_classify: add filter dnn_classify for classification based on detection bounding boxes

2021-04-18 Thread Guo, Yejun
classification is done on every detection bounding box in frame's side data, which are the results of object detection (filter dnn_detect). Please refer to commit log of dnn_detect for the material for detection, and see below for classification. - download material for classifcation: wget https

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Gyan Doshi
On 2021-04-18 15:25, Guo, Yejun wrote: -Original Message- From: ffmpeg-devel On Behalf Of Gyan Doshi Sent: 2021年4月18日 13:04 To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command Ok. Will apply. On 2021-04-18 10:11, G

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Gyan Doshi
On 2021-04-18 17:01, Gyan Doshi wrote: On 2021-04-18 15:25, Guo, Yejun wrote: -Original Message- From: ffmpeg-devel On Behalf Of Gyan Doshi Sent: 2021年4月18日 13:04 To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

Re: [FFmpeg-devel] [PATCH 3/4] avformat/rmdec: Use 64bit for intermediate for DEINT_ID_INT4

2021-04-18 Thread Michael Niedermayer
On Fri, Apr 16, 2021 at 08:37:51PM -0300, James Almer wrote: > On 4/16/2021 7:45 PM, James Almer wrote: > > On 4/16/2021 7:24 PM, Andreas Rheinhardt wrote: > > > James Almer: > > > > On 4/16/2021 4:04 PM, Michael Niedermayer wrote: > > > > > On Thu, Apr 15, 2021 at 06:22:10PM -0300, James Almer wro

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: private_stream_1 is not asynchronous KLV

2021-04-18 Thread Marton Balint
On Sat, 17 Apr 2021, Mao Hata wrote: According to ISO/IEC 13818-1, private_stream_1 is a synchronous (has PTS/DTS) stream. Asynchronous one is private_stream_2. Which section describes this? Also keep in mind that code was added so that AV_CODEC_ID_SMPTE_KLV is handled in mpegts, and SMPT

Re: [FFmpeg-devel] [PATCH v1 1/3] libavformat/rtsp.h : Add member ignore_rtcp to RTSPState

2021-04-18 Thread Moritz Barsnick
On Sun, Apr 18, 2021 at 00:55:26 +0800, su...@epoint.com.cn wrote: > From: Minlei Sun > > Signed-off-by: Minlei Sun These three patches can certainly be squashed - they are of no use individually. And you need to amend the documentation (doc/protocols.texi). Also, I'm not sure whether ignore_r

[FFmpeg-devel] [ffmpeg-web][PATCH] Drop my entry from the consulting list

2021-04-18 Thread Stefano Sabatini
Hi, I've not been really active in the last period and this patch clarifies this condition. Best regards, Stefano >From 38bc43827b9a6b2d967514323a718abee7e7211d Mon Sep 17 00:00:00 2001 From: Stefano Sabatini Date: Sun, 18 Apr 2021 22:23:20 +0200 Subject: [PATCH] consulting: remove my entry from

Re: [FFmpeg-devel] [PATCH] avutil/cpu: Use HW_NCPUONLINE to detect # of online CPUs with OpenBSD

2021-04-18 Thread Marton Balint
On Fri, 16 Apr 2021, Brad Smith wrote: ping. Will apply, thanks. Marton On 4/3/2021 2:49 PM, Brad Smith wrote: avutil/cpu: Use HW_NCPUONLINE to detect # of online CPUs with OpenBSD Signed-off-by: Brad Smith --- libavutil/cpu.c | 6 ++ 1 file changed, 6 insertions(+) diff --gi

[FFmpeg-devel] [PATCH 1/2] lavf/avio: add avio_vprintf()

2021-04-18 Thread Stefano Sabatini
This new function makes it possible to use avio_printf() functionality from a function taking a variable list of arguments. --- doc/APIchanges| 3 +++ libavformat/avio.h| 6 ++ libavformat/aviobuf.c | 17 + libavformat/version.h | 2 +- 4 files changed, 23 insert

[FFmpeg-devel] [PATCH 2/2] ffprobe: add -o option

2021-04-18 Thread Stefano Sabatini
This enables printing to a resource specified with -o OUTPUT. Address issue: http://trac.ffmpeg.org/ticket/8024 --- doc/ffprobe.texi | 7 ++ fftools/ffprobe.c | 174 ++ 2 files changed, 120 insertions(+), 61 deletions(-) diff --git a/doc/ffprobe.tex

Re: [FFmpeg-devel] [PATCH v2] avformat/wavdec: Fix reading files with id3v2 apic before fmt tag

2021-04-18 Thread Andreas Rheinhardt
Paul B Mahol: > Why you put nonsense regression part in log. > > That is very unfriendly behavior from you. > > Noted. > Before your patch, the audio in the files in question would work; with your patch it doesn't any longer. That's a regression. > On Sun, Apr 18, 2021 at 1:50 AM Andreas Rhein

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Guo, Yejun
> -Original Message- > From: ffmpeg-devel On Behalf Of > Gyan Doshi > Sent: 2021年4月18日 23:45 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for > unknown command > > > > On 2021-04-18 17:01, Gyan Doshi wrote: > > > > On 2021-04-18

[FFmpeg-devel] [PATCH 1/7] avcodec/avcodec: Actually honour the documentation of subtitle_header

2021-04-18 Thread Andreas Rheinhardt
It is only supposed to be freed by libavcodec for decoders, yet avcodec_open2() always frees it on failure. Furthermore, avcodec_close() doesn't free it for decoders. Both of this has been changed. Signed-off-by: Andreas Rheinhardt --- This might be squashed with the next patch. libavcodec/avco

[FFmpeg-devel] [PATCH 2/7] avcodec/avcodec: Document current behaviour of subtitle_header

2021-04-18 Thread Andreas Rheinhardt
Mention that avcodec_free_context() always frees it even when encoding. And mention that freeing is of course performed in avcodec_close() when decoding and not necessarily in avcodec_open2(). Signed-off-by: Andreas Rheinhardt --- libavcodec/avcodec.h | 4 +++- 1 file changed, 3 insertions(+), 1

[FFmpeg-devel] [PATCH 3/7] avcodec/avcodec: Remove unnecessary forward declaration

2021-04-18 Thread Andreas Rheinhardt
Forgotten in ba6cada92eb7c3446bfb1d4525031d405a052516. Signed-off-by: Andreas Rheinhardt --- libavcodec/avcodec.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 9b68aecd31..b9b487be41 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcod

[FFmpeg-devel] [PATCH 5/7] avcodec/avcodec: Store whether AVCodec->close needs to be called

2021-04-18 Thread Andreas Rheinhardt
Right now all AVCodecContexts except those using frame-threaded decoding call the codec's init function and expect its close function to be called. In order to make sure that the close function is not called for frame-threaded decoding ff_frame_thread_free() resets AVCodecContext.codec (and because

[FFmpeg-devel] [PATCH 6/7] avcodec/avcodec: Use avcodec_close() on avcodec_open2() failure

2021-04-18 Thread Andreas Rheinhardt
Compared to the earlier behaviour the following changes: a) AVCodecInternal.byte_buffer is freed. b) The last_pkt_props FIFO is emptied before freeing it. c) If set AVCodecContext.hwaccel is uninitialized and its private data is freed; hw_frames_ctx and hw_device_ctx are also unreferenced. d) coded

[FFmpeg-devel] [PATCH 4/7] avcodec/avcodec: Free frame_thread_encoder on avcodec_open2() error

2021-04-18 Thread Andreas Rheinhardt
The frame_thread_encoder has so far not been freed in case an error happened in avcodec_open2() after ff_frame_thread_encoder_init(). This commit changes this. Signed-off-by: Andreas Rheinhardt --- libavcodec/avcodec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/avcodec.c b/

[FFmpeg-devel] [PATCH 7/7] avcodec/exr: Return correct error code on allocation failure

2021-04-18 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/exr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/exr.c b/libavcodec/exr.c index 49c4774145..4f55609b0c 100644 --- a/libavcodec/exr.c +++ b/libavcodec/exr.c @@ -2245,7 +2245,7 @@ static av_cold int decode_init(AVC

Re: [FFmpeg-devel] [PATCH v3 1/1] avformat/mpegtsenc: Write necessary descriptors into PMT for arib_caption muxing

2021-04-18 Thread zheng qian
Is there anyone who could review this patch? Best regards, zheng ___ 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 subje

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Andriy Gelman
Hi, On Mon, 19. Apr 01:10, Guo, Yejun wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Gyan Doshi > > Sent: 2021年4月18日 23:45 > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for > > unknown command > > > >

Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown command

2021-04-18 Thread Gyan Doshi
On 2021-04-19 11:02, Andriy Gelman wrote: Hi, On Mon, 19. Apr 01:10, Guo, Yejun wrote: -Original Message- From: ffmpeg-devel On Behalf Of Gyan Doshi Sent: 2021年4月18日 23:45 To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] doc/muxers.texi: fix build issue for unknown c