Re: [FFmpeg-devel] [PATCH] fate: add more tests for hue video filter

2018-05-11 Thread Tobias Rapp
On 10.05.2018 23:04, Michael Niedermayer wrote: On Wed, May 09, 2018 at 12:05:48PM +0200, Tobias Rapp wrote: Adds tests for the hue angle and brightness filter parameters. Renames the existing saturation parameter test for consistency. Signed-off-by: Tobias Rapp --- tests/fate/filter-video.m

[FFmpeg-devel] [PATCH] qsv: Use the installed mfx include headers if possibile

2018-05-11 Thread Haihao Xiang
Currently an extra copy of mfx include headers from {MSDK_INSTALL_PREFIX}/include to {MSDK_INSTALL_PREFIX}/include/mfx is required when using pkg-config for libmfx detection. This fix checks the installed mfx include headers first, and falls back to the legacy way if that fails Signed-off-by: Hai

Re: [FFmpeg-devel] [PATCH 3/4] lavc/h2645_parse: add h264_nal_unit_name for h264 NAL type.

2018-05-11 Thread Steven Liu
> On May 11, 2018, at 13:11, Jun Zhao wrote: > > Signed-off-by: Jun Zhao > --- > libavcodec/h2645_parse.c | 26 -- > 1 file changed, 24 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c > index 579b2c9..8d67579 100644 > -

Re: [FFmpeg-devel] [FFmpeg-devel, RFC] lavfi: add opencl tonemap filter.

2018-05-11 Thread Niklas Haas
> The way my scene change detection heuristic works like this: I trigger a > scene change (and therefore discard the frame averaging buffer) if the > distance between the current frame average brightness and the current > running average exceeds a threshold value, that threshold being (by > default

Re: [FFmpeg-devel] [RFC] On behaviour of unlabelled outputs in filtergraphs.

2018-05-11 Thread Nicolas George
Gyan Doshi (2018-05-09): > In the open ticket 7089, I reported that (terminal) filters with unlabelled > output pads are automatically forwarded to the first output file and ordered > before any expressly mapped streams. There's no warning issued. > > Is this behaviour intended? Short answer: yes

Re: [FFmpeg-devel] [PATCH 1/4] lavc/h2645_parse: log more HEVC NAL type.

2018-05-11 Thread Mark Thompson
On 11/05/18 06:11, Jun Zhao wrote: > Signed-off-by: Jun Zhao > --- > libavcodec/h2645_parse.c | 16 > 1 file changed, 16 insertions(+) > > diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c > index dbf2435..ec25c1e 100644 > --- a/libavcodec/h2645_parse.c > +++ b/li

Re: [FFmpeg-devel] [PATCH 4/4] lavc/cbs_h2645: fix no slice data trigger the assert.

2018-05-11 Thread Mark Thompson
On 11/05/18 06:11, Jun Zhao wrote: > when the NALU data with zero, just give a warning. > > Fixes ticket #7200 > > Signed-off-by: Jun Zhao > --- > libavcodec/cbs_h2645.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.

Re: [FFmpeg-devel] [FFmpeg-devel, RFC] lavfi: add opencl tonemap filter.

2018-05-11 Thread Moritz Barsnick
On Fri, May 11, 2018 at 10:10:42 +0200, Niklas Haas wrote: > Thinking about this logic again, I came to realize that a different > strategy might be to check instead for a minimum threshold brightness > difference in a critical number of different areas of the screen. This > way, a very bright ligh

Re: [FFmpeg-devel] [PATCH 3/4] lavc/h2645_parse: add h264_nal_unit_name for h264 NAL type.

2018-05-11 Thread Mark Thompson
On 11/05/18 06:11, Jun Zhao wrote: > Signed-off-by: Jun Zhao > --- > libavcodec/h2645_parse.c | 26 -- > 1 file changed, 24 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c > index 579b2c9..8d67579 100644 > --- a/libavcode

Re: [FFmpeg-devel] [PATCH] qsv: Use the installed mfx include headers if possibile

2018-05-11 Thread Mark Thompson
On 11/05/18 08:52, Haihao Xiang wrote: > Currently an extra copy of mfx include headers from > {MSDK_INSTALL_PREFIX}/include > to {MSDK_INSTALL_PREFIX}/include/mfx is required when using pkg-config for > libmfx detection. This fix checks the installed mfx include headers first, > and falls back to

Re: [FFmpeg-devel] [PATCH] qsv: Use the installed mfx include headers if possibile

2018-05-11 Thread wm4
On Fri, 11 May 2018 11:38:52 +0100 Mark Thompson wrote: > On 11/05/18 08:52, Haihao Xiang wrote: > > Currently an extra copy of mfx include headers from > > {MSDK_INSTALL_PREFIX}/include > > to {MSDK_INSTALL_PREFIX}/include/mfx is required when using pkg-config for > > libmfx detection. This fix

Re: [FFmpeg-devel] [PATCH 3/5] ffserver: Implement http interface and implementation

2018-05-11 Thread Moritz Barsnick
On Thu, May 10, 2018 at 17:41:24 +0200, Stephan Holljes wrote: > +while ((handshake = avio_handshake(client_ctx)) > 0) { > +av_opt_get(client_ctx, "method", AV_OPT_SEARCH_CHILDREN, &method); > +av_opt_get(client_ctx, "resource", AV_OPT_SEARCH_CHILDREN, > &resource); Cast the l

Re: [FFmpeg-devel] [PATCH 4/5] ffserver: Implement ffserver and add Makefile

2018-05-11 Thread Moritz Barsnick
On Thu, May 10, 2018 at 17:41:25 +0200, Stephan Holljes wrote: > +ffserver: segment.o publisher.o lavfhttpd.o ffserver.c > + cc -g -Wall $(LAV_FLAGS) -lpthread -o ffserver segment.o publisher.o > lavfhttpd.o ffserver.c ^ $(CC) $(CFLAGS) to be more generic and allow overriding. The LDF

Re: [FFmpeg-devel] [PATCH 3/5] ffserver: Implement http interface and implementation

2018-05-11 Thread Stephan Holljes
On Fri, May 11, 2018 at 2:39 PM, Moritz Barsnick wrote: > On Thu, May 10, 2018 at 17:41:24 +0200, Stephan Holljes wrote: >> +while ((handshake = avio_handshake(client_ctx)) > 0) { >> +av_opt_get(client_ctx, "method", AV_OPT_SEARCH_CHILDREN, &method); >> +av_opt_get(client_ctx,

Re: [FFmpeg-devel] [PATCH 5/5] ffserver: Add basic documentation of the architecture

2018-05-11 Thread Moritz Barsnick
On Thu, May 10, 2018 at 17:41:26 +0200, Stephan Holljes wrote: > +In its current for this is a HTTP live-streaming server. A media file can be ^ form^ an > +streamed to a number of clients with the correct media file being muxed on- > +the-fly for each client starting at th

Re: [FFmpeg-devel] [PATCH 4/5] ffserver: Implement ffserver and add Makefile

2018-05-11 Thread Moritz Barsnick
On Fri, May 11, 2018 at 14:49:53 +0200, Moritz Barsnick wrote: > [matroska,webm @ 0xa166580] Finding video stream. > [AVIOContext @ 0xb441e5c0] Shutting down http server. > > "Shutting down"??? And I can't connect and play. Forget this part, my static video was too short, and I didn't realize it

Re: [FFmpeg-devel] [PATCH 4/5] ffserver: Implement ffserver and add Makefile

2018-05-11 Thread Stephan Holljes
On Fri, May 11, 2018 at 2:49 PM, Moritz Barsnick wrote: > On Thu, May 10, 2018 at 17:41:25 +0200, Stephan Holljes wrote: >> +ffserver: segment.o publisher.o lavfhttpd.o ffserver.c >> + cc -g -Wall $(LAV_FLAGS) -lpthread -o ffserver segment.o publisher.o >> lavfhttpd.o ffserver.c > ^ $

Re: [FFmpeg-devel] [RFC] On behaviour of unlabelled outputs in filtergraphs.

2018-05-11 Thread Gyan Doshi
On 5/11/2018 3:08 PM, Nicolas George wrote: Note that the behaviour when explicit -map options are present: when they are, automatic selection of "best" input streams is disabled, and it would seem logical that automatic selection of unlabelled filter outputs would be disabled too. I would not

Re: [FFmpeg-devel] [PATCH 5/5] ffserver: Add basic documentation of the architecture

2018-05-11 Thread Stephan Holljes
On Fri, May 11, 2018 at 2:54 PM, Moritz Barsnick wrote: > On Thu, May 10, 2018 at 17:41:26 +0200, Stephan Holljes wrote: >> +In its current for this is a HTTP live-streaming server. A media file can be > ^ form^ an > >> +streamed to a number of clients with the correct medi

[FFmpeg-devel] [PATCH] avfilter: add bm3d filter

2018-05-11 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile |1 + libavfilter/allfilters.c |1 + libavfilter/vf_bm3d.c| 1002 ++ 3 files changed, 1004 insertions(+) create mode 100644 libavfilter/vf_bm3d.c diff --git a/libavfilter/Makefile b/

Re: [FFmpeg-devel] [RFC] On behaviour of unlabelled outputs in filtergraphs.

2018-05-11 Thread Nicolas George
Gyan Doshi (2018-05-11): > Other points noted, but I still think this should throw an error and > exit_program. Explicit inclusions should not be mixed with implicit ones, > given how map is described to the user. On principle, I think I agree. But since it is already working differently, I think

Re: [FFmpeg-devel] [PATCH] Support for Ambisonics and OpusProjection* API.

2018-05-11 Thread Drew Allen
Hi Rostislav et all, The IETF document has just been moved to a working group last call. Do you think it would it be possible to land this patch under an experimental flag? Cheers, Drew On Mon, Apr 23, 2018 at 12:28 PM Rostislav Pehlivanov wrote: > On 23 April 2018 at 17:02, Drew Allen > wro

Re: [FFmpeg-devel] [PATCH 4/4] lavc/cbs_h2645: fix no slice data trigger the assert.

2018-05-11 Thread James Almer
On 5/11/2018 7:10 AM, Mark Thompson wrote: > On 11/05/18 06:11, Jun Zhao wrote: >> when the NALU data with zero, just give a warning. >> >> Fixes ticket #7200 >> >> Signed-off-by: Jun Zhao >> --- >> libavcodec/cbs_h2645.c | 6 +- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --

Re: [FFmpeg-devel] [PATCH] Support for Ambisonics and OpusProjection* API.

2018-05-11 Thread Rostislav Pehlivanov
On 11 May 2018 at 15:55, Drew Allen wrote: > Hi Rostislav et all, > > The IETF document has just been moved to a working group last call. > > Do you think it would it be possible to land this patch under an > experimental flag? No, libopus itself still hides the new API under an experimental fl

[FFmpeg-devel] [PATCH] avfilter/vf_nlmeans: add parameter to control weight of centered pixel

2018-05-11 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_nlmeans.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libavfilter/vf_nlmeans.c b/libavfilter/vf_nlmeans.c index 82e779ce85..872fa90e28 100644 --- a/libavfilter/vf_nlmeans.c +++ b/libavfilter/vf_nlmeans.c @@ -52,6 +5

Re: [FFmpeg-devel] [PATCH] lavc/amfenc: moving amf common code (library and context) to lavu/hwcontext_amf from amfenc to be reused in other amf components

2018-05-11 Thread Alexander Kravchenko
Hi Mark, Thank you for your comments. Could you see my comments and questions bellow > -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Mark > Thompson > Sent: Thursday, May 10, 2018 11:43 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpe

Re: [FFmpeg-devel] [PATCH 10/13] avformat/mxfenc: Set color siting to 0 for D10-MXF

2018-05-11 Thread Michael Niedermayer
On Tue, May 08, 2018 at 07:41:33PM +0200, Michael Niedermayer wrote: > On Tue, May 08, 2018 at 12:55:18PM +0200, Tomas Härdin wrote: > > mån 2018-05-07 klockan 12:38 +0200 skrev Michael Niedermayer: > > > > Signed-off-by: Michael Niedermayer > > > --- > > >  libavformat/mxfenc.c   

Re: [FFmpeg-devel] [PATCH] avfilter: add bm3d filter

2018-05-11 Thread Michael Niedermayer
On Fri, May 11, 2018 at 04:03:07PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile |1 + > libavfilter/allfilters.c |1 + > libavfilter/vf_bm3d.c| 1002 > ++ > 3 files changed, 1004 insertions(+) >