Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-09 9:59 GMT+08:00 Aman Gupta : > Yea this doesn't work at all. I wish someone had bothered to check before > the patch was proposed or merged. > > With "-ss 20 -i file -segment_time 2 -segment_start_number 10 > -initial_offset 20": > > #EXTM3U > #EXT-X-VERSION:3 > #EXT-X-MEDIA-SEQUENCE:0 >

[FFmpeg-devel] [PATCH v4] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
v4 : - fix sharpless typo(sharpless -> sharpness) - when don't support denoise/sharpness, report the error and return - fix denoise/sharpness params buffer leak in error handle v3 : fix sharpless mapping issue v2 : fix filter support flag check logic issue From d1f0b556bdd6be4dffcd3b1b93c

[FFmpeg-devel] Fwd: Re: [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
cc Mark Forwarded Message Subject: Re: [FFmpeg-devel] [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support Date: Mon, 5 Sep 2016 09:52:22 +0800 From: Jun Zhao To: FFmpeg development discussions and patches On 2016/8/31 6:48, Mark Thompson wrote: > On 30/08/16 09:

[FFmpeg-devel] Fwd: Re: [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
cc Mark Forwarded Message Subject: Re: [FFmpeg-devel] [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support Date: Mon, 5 Sep 2016 09:33:43 +0800 From: Jun Zhao To: ffmpeg-devel@ffmpeg.org On 2016/8/31 6:48, Mark Thompson wrote: > On 30/08/16 09:00, Jun Zhao wrote: >

[FFmpeg-devel] [PATCH] avformat/segment: fix the duration error of use output_ts_offset

2016-09-08 Thread Steven Liu
This patch can merge with 1da00be009aa74400042bf470b9a5ffbd82a1c5e i have checked this modify: ./ffmpeg -i ~/facebook.mp4 -c copy -f segment -segment_time 2 -output_ts_offset 80 -segment_list output-test.m3u8 -v debug output-test-%03d.ts #EXTM3U #EXT-X-VERSION:3 #EXT-X-MEDIA-SEQUENCE:0 #EXT-X-ALL

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable stream parsing for VP9.

2016-09-08 Thread Ronald S. Bultje
Hi, On Wed, Sep 7, 2016 at 7:37 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, Sep 6, 2016 at 10:39 PM, Matthew Gregan wrote: > >> At 2016-09-06T22:18:18-0400, Ronald S. Bultje wrote: >> > I think the patch is fine, but I wonder if it should set it to _FULL >> (the >> > parser ignores the option

Re: [FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread Ronald S. Bultje
Hi, On Thu, Sep 8, 2016 at 9:50 PM, James Almer wrote: > On 9/8/2016 8:41 PM, Alex Converse wrote: > > Fixes Netflix VP9 DASH samples. > > --- > > libavformat/mov.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavformat/mov.c b/libavformat/mov.c > > index a7595c5..6e80b93 10

Re: [FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread James Almer
On 9/8/2016 8:41 PM, Alex Converse wrote: > Fixes Netflix VP9 DASH samples. > --- > libavformat/mov.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index a7595c5..6e80b93 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -2161,6 +21

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
Yea this doesn't work at all. I wish someone had bothered to check before the patch was proposed or merged. With "-ss 20 -i file -segment_time 2 -segment_start_number 10 -initial_offset 20": #EXTM3U #EXT-X-VERSION:3 #EXT-X-MEDIA-SEQUENCE:0 #EXT-X-ALLOW-CACHE:YES #EXT-X-TARGETDURATION:3 #EXTINF:2.

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
I tried to switch from -initial_offset to -output_ts_offset, but it does not work as expected. In Safari, the generated HLS stream stalls out and stops playing after the first segment. Aman On Thu, Sep 8, 2016 at 5:54 PM, Michael Niedermayer wrote: > On Fri, Sep 09, 2016 at 02:59:52AM +0800, St

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
On Thu, Sep 8, 2016 at 9:18 AM, Carl Eugen Hoyos wrote: > 2016-09-08 15:48 GMT+02:00 Aman Gupta : > > On Thursday, September 8, 2016, Carl Eugen Hoyos > wrote: > > > >> 2016-09-08 10:19 GMT+02:00 Aman Gupta >: > >> > >> > +{ "a53cc", "Use A53 Closed Captions (if available)", > >> OFFSET(a53_

Re: [FFmpeg-devel] [PATCH 2/2] Normalize VP8 and VP9 tags when writing IVF.

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:41:03PM -0700, Alex Converse wrote: > VP9-in-ISOM uses vp08 and vp09 tags, while ivf uses VP80 and VP90. > --- > libavformat/ivfenc.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c > ind

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Michael Niedermayer
On Fri, Sep 09, 2016 at 02:59:52AM +0800, Steven Liu wrote: > 2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > > > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > > > +if (seg->initial_offset > 0) { > > > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will > > be > > > deprec

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add myself for hlsenc

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 12:56:19AM +0800, Steven Liu wrote: > Signed-off-by: Steven Liu > --- > MAINTAINERS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied maybe the entry should be split between encryption for hlsenc and the rest of hlsenc thx [...] -- Michael GnuPG fing

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:18:26PM +0200, Matthieu Bouron wrote: > On Thu, Sep 8, 2016 at 2:28 PM, Michael Niedermayer > wrote: > > > On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > > --- > > > libavcodec/hevc.h| 9 + > > > lib

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:49:12PM +0300, Ligverd Haer wrote: > > Please send your updated patch as attachment to a > > reply in this mailing list thread, > > > > Please tell us if documentation pointed you to github or trac to > > post patches, we try hard to have patch submission (only) on > > t

[FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread Alex Converse
Fixes Netflix VP9 DASH samples. --- libavformat/mov.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index a7595c5..6e80b93 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2161,6 +2161,7 @@ static int mov_finalize_stsd_codec(MOVContext *c, AVI

[FFmpeg-devel] [PATCH 2/2] Normalize VP8 and VP9 tags when writing IVF.

2016-09-08 Thread Alex Converse
VP9-in-ISOM uses vp08 and vp09 tags, while ivf uses VP80 and VP90. --- libavformat/ivfenc.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c index 5dbcd97..1735606 100644 --- a/libavformat/ivfenc.c +++ b/libavformat/ivfen

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Sven C. Dack wrote: On 08/09/16 15:17, Andy Furniss wrote: Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anything relate

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/alsdec: Fix reading 0 mantisse bits

2016-09-08 Thread James Almer
> ffmpeg | branch: master | Michael Niedermayer | > Thu Sep 8 22:02:44 2016 +0200| [037422178d7f1d0dd09e1ce424dd61a69e77668b] | > committer: Michael Niedermayer > > avcodec/alsdec: Fix reading 0 mantisse bits > > Fixes assertion failure > Fixes: > 848c24abc1721c9e3d1ba7bfee8d9fcc/asan_heap-oob

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Rodger Combs
> On Sep 8, 2016, at 07:49, Nicolas George wrote: > > Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : >> --- >> ffmpeg.c | 6 +- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/ffmpeg.c b/ffmpeg.c >> index d858407..1d793fe 100644 >> --- a/ffmpeg.c >> +++ b/ffmpe

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > +if (seg->initial_offset > 0) { > > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will > be > > deprecated soon," > > "is deprecated" even if you decide not to ignore the value s

[FFmpeg-devel] [PATCH] avcodec/mlz: Check output chars before using it

2016-09-08 Thread Michael Niedermayer
Fixes hypothetical integer overflow Signed-off-by: Michael Niedermayer --- libavcodec/mlz.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/libavcodec/mlz.c b/libavcodec/mlz.c index 039635d..a2d1b89 100644 --- a/libavcodec/mlz.c +++ b/libavcodec/mlz.c @

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Nicolas George
Le tridi 23 fructidor, an CCXXIV, Carl Eugen Hoyos a écrit : > xcb is slower? Looking at the code, I see that xcbgrab allocates and frees the shared memory segment for each frame, while x11grab only allocates it once at the beginning. xcbgrab does that so that it can wrap the segment in a refcount

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 19:16 GMT+02:00 Andy Furniss : It is a bit faster xcb is slower? Yes (unless I am mixing things up) --disable-libxcb --enable-x11grab is faster than autodetect ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 19:16 GMT+02:00 Andy Furniss : > It is a bit faster xcb is slower? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 16:17 GMT+02:00 Andy Furniss : Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anythin

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-09-08 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 07:25:31PM +0530, Umair Khan wrote: > On Sun, Jul 17, 2016 at 1:50 PM, Thilo Borgmann > wrote: > > Hi, > > > >> From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 > >> From: Umair Khan > >> Date: Sun, 17 Jul 2016 13:05:49 +0530 > >> Subject: [PATCH 1/1

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:48 GMT+02:00 Aman Gupta : > On Thursday, September 8, 2016, Carl Eugen Hoyos wrote: > >> 2016-09-08 10:19 GMT+02:00 Aman Gupta >: >> >> > +{ "a53cc", "Use A53 Closed Captions (if available)", >> OFFSET(a53_cc), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VE }, >> >> Why is this disabled

Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext - fontsize

2016-09-08 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 10:27:24AM -0700, Brett Harrison wrote: > This patch addresses your concerns. > > On Fri, Sep 2, 2016 at 5:05 PM, Michael Niedermayer > wrote: > > > On Fri, Sep 02, 2016 at 03:31:21PM -0700, Brett Harrison wrote: > > > Addressed the following concerns. > > > > > > === > >

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
On Thursday, September 8, 2016, Carl Eugen Hoyos wrote: > 2016-09-08 10:19 GMT+02:00 Aman Gupta >: > > > +{ "a53cc", "Use A53 Closed Captions (if available)", > OFFSET(a53_cc), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VE }, > > Why is this disabled by default? I copied this from the libx264 and

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:52 GMT+02:00 Sven C. Dack : > X11grab is here only the input device. Of course. I know that and I assume Andy also knows. Still Andy claims that (our) xcb input is slower than GStreamer's x11 input and I would like him to test if the reason is our change to xcb. Carl Eugen ___

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Sven C. Dack
On 08/09/16 15:17, Andy Furniss wrote: Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anything related with configure. I

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:01 GMT+02:00 Steven Liu : > +if (seg->initial_offset > 0) { > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will be > deprecated soon," "is deprecated" even if you decide not to ignore the value set. Carl Eugen _

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:17 GMT+02:00 Andy Furniss : > Carl Eugen Hoyos wrote: >> >> 2016-09-08 12:01 GMT+02:00 Andy Furniss : >> >>> I don't know what it is about x11grab/CSC with ffmpeg, but >>> on my old CPU gstreamer is twice as fast. >> >> x11grab or xcb? > > I guess xcb as I don't --enable anything rela

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Matthieu Bouron
On Thu, Sep 8, 2016 at 2:28 PM, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > --- > > libavcodec/hevc.h| 9 + > > libavcodec/hevc_ps.c | 27 +++ > > 2 files changed, 36 inserti

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anything related with configure. _

Re: [FFmpeg-devel] [RFC] marking non developer tickets

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 03:23:36PM +0200, Carl Eugen Hoyos wrote: > 2016-09-07 13:31 GMT+02:00 Michael Niedermayer : > > > Can type = task be used for non devel tasks > > No objections although I don't understand how #5832 can be "fixed" > and I don't like tickets that by definition are unfixable

[FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
ffmpeg have a generic solution working with all muxer named output_ts_offset, output_ts_offset will instead of initial_offset Signed-off-by: LiuQi --- libavformat/segment.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/segment.c b/libavformat/segment.c index bf29ef8..23e1a

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
> Please send your updated patch as attachment to a > reply in this mailing list thread, > > Please tell us if documentation pointed you to github or trac to > post patches, we try hard to have patch submission (only) on > this mailing list. > > Carl Eugen diff --git a/libavformat/dashenc.c b/li

Re: [FFmpeg-devel] [PATCH 2/2] doc/muxers: remove initial_offset from segment muxer

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:33 GMT+02:00 Steven Liu : > -@item initial_offset @var{offset} > -Specify timestamp offset to apply to the output packet timestamps. The > -argument must be a time duration specification, and defaults to 0. This should be merged with the other patch. (If the option gets deprecated it

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: remove initial_offset option from segment.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:33 GMT+02:00 Steven Liu : > -{ "initial_offset", "set initial timestamp offset", > OFFSET(initial_offset), AV_OPT_TYPE_DURATION, > {.i64 = 0}, -INT64_MAX, INT64_MAX, E }, You generally cannot simply remove options from FFmpeg. (I did not look if the option is very new or imposes

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:34 GMT+02:00 Ligverd Haer : > В письме от четверг, 8 сентября 2016 г. 14:35:27 MSK пользователь Michael > Niedermayer написал: > >> patches should be sent to the mailing list, patches on trac tend to >> be missed and forgotten > > Don't want to clog the ffmpeg-devel mailing list. Pl

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
В письме от четверг, 8 сентября 2016 г. 14:35:27 MSK пользователь Michael Niedermayer написал: > patches should be sent to the mailing list, patches on trac tend to > be missed and forgotten Don't want to clog the ffmpeg-devel mailing list. I made a pull request on github, I was sent to ffmpeg-

[FFmpeg-devel] [PATCH 2/2] doc/muxers: remove initial_offset from segment muxer

2016-09-08 Thread Steven Liu
Signed-off-by: LiuQi --- doc/muxers.texi | 4 1 file changed, 4 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index c39a6a0..4fd4133 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -1355,10 +1355,6 @@ will start with near-zero timestamps. It is meant to ease the playback

[FFmpeg-devel] [PATCH 1/2] avformat/segment: remove initial_offset option from segment.c

2016-09-08 Thread Steven Liu
ffmpeg have a generic solution working with all muxer named output_ts_offset Signed-off-by: LiuQi --- libavformat/segment.c | 18 ++ 1 file changed, 2 insertions(+), 16 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index bf29ef8..2fc380e 100644 --- a/li

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 10:19 GMT+02:00 Aman Gupta : > +{ "a53cc", "Use A53 Closed Captions (if available)", OFFSET(a53_cc), > AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VE }, Why is this disabled by default? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

Re: [FFmpeg-devel] [RFC] marking non developer tickets

2016-09-08 Thread Carl Eugen Hoyos
2016-09-07 13:31 GMT+02:00 Michael Niedermayer : > Can type = task be used for non devel tasks No objections although I don't understand how #5832 can be "fixed" and I don't like tickets that by definition are unfixable. I regularly look at -random. Carl Eugen __

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 12:01 GMT+02:00 Andy Furniss : > I don't know what it is about x11grab/CSC with ffmpeg, but > on my old CPU gstreamer is twice as fast. x11grab or xcb? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailma

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 08:42:05PM +0800, Steven Liu wrote: > 2016-09-08 20:31 GMT+08:00 Michael Niedermayer : > > > On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > > > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu > > wrote: > > > > > > > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] [RFC] libavfilter HRTF/room acoustics filter [2/2]

2016-09-08 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 06:18:33AM -0700, Yue Shi Lai wrote: > This is 2/2 of the patch. > af_hrtf_mitkemar.c |14076 > + > 1 file changed, 14076 insertions(+) > c4c72f78adf6aebf20ccd1883388bb69d07b9bf9 > 0002-HRTF-filter-ported-from-MPlayer-

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Nicolas George
Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : > --- > ffmpeg.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index d858407..1d793fe 100644 > --- a/ffmpeg.c > +++ b/ffmpeg.c > @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Steven Liu
2016-09-08 20:31 GMT+08:00 Michael Niedermayer : > On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu > wrote: > > > > > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer >: > > > > > > > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wr

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 07:17:59PM -0500, Rodger Combs wrote: > --- > ffmpeg.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) ill leave these 2 patches for nicolas to review ... thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically f

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 09:58:02AM +0300, Ligverd Haer wrote: > > > Common attributes for AdaptationSet and Representation shall either be in > > > one of the elements but not in both. > > > > > > in particular, the attribute frameRate is duplicated in the node > > > Representation > > > > There

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu wrote: > > > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > > > > > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > > > From: Aman Gupta > > > > > > > > --- > > > > doc/m

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavcodec/hevc.h| 9 + > libavcodec/hevc_ps.c | 27 +++ > 2 files changed, 36 insertions(+) > > diff --git a/libavcodec/hevc.h b/libavcodec/hevc.h > index be9

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Aman Gupta
On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu wrote: > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > > > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > > From: Aman Gupta > > > > > > --- > > > doc/muxers.texi | 4 > > > libavformat/hlsenc.c | 13 +++-- > > >

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Sven C. Dack
Just a guess, but you are using nv_surface->height, which appears to have been rounded up to 1088 while the actual size is 1080. The 1088 is then passed on to av_image_copy() where it probably tries to read 1920x1088 pixels from the source, but the source might not have that much memory and so i

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_headers_first option

2016-09-08 Thread Stefano Sabatini
On date Sunday 2016-09-04 19:09:32 +0200, Nicolas George encoded: > Le nonidi 19 fructidor, an CCXXIV, Stefano Sabatini a écrit : > > This is meant to be used for generating output suitable for the > > ffprobe_default demuxer. > > --- > > ffprobe.c | 30 ++ > > 1 file c

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7 inse

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Sven C. Dack wrote: It is not an issue. x11grab produces BGR0 and nvenc can handle it with the patch. It's giving me 100fp/s (up from 47fp/s) with a 1920x1080 monitor. I'd imagine people with 4K displays will be happy, too, although they will have to live with lower speeds of perhaps 30 fp/s. Wo

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Sven C. Dack
Hallo, ich schicke Dir noch einen weitere Backtrace mit etwas mehr Details: (gdb) bt #0 0x7f19eb8e3a3e in ?? () from /lib/x86_64-linux-gnu/libc.so.6 #1 0x00fbc63e in av_image_copy_plane (height=8, bytewidth=7680, src_linesize=, src=0x7f19ea08f000 "", dst_linesize=out>, dst=) at sr

Re: [FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-08 Thread Ronald S. Bultje
Hi Andrew, On Thu, Sep 8, 2016 at 3:59 AM, qw wrote: > I have made one simple transcoding program by using ffmpeg lib, where > several av transcoding tasks are done in one process and each task > corresponds to one thread. > > But some ffmpeg function will report error, and sometimes the multi-t

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Steven Liu
2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > From: Aman Gupta > > > > --- > > doc/muxers.texi | 4 > > libavformat/hlsenc.c | 13 +++-- > > 2 files changed, 15 insertions(+), 2 deletions(-) > > Isnt this red

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > From: Aman Gupta > > --- > doc/muxers.texi | 4 > libavformat/hlsenc.c | 13 +++-- > 2 files changed, 15 insertions(+), 2 deletions(-) Isnt this redundant with -output_ts_offset ? or how do they differ ? [...] --

Re: [FFmpeg-devel] [PATCH] Fix potential integer overflow in mov_read_keys

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 02:38:48PM -0700, Sergey Volk wrote: > I just realized that count+1 itself might overflow if count==UINT_MAX, so I > guess it's better to subtract 1 from the right-hand side. Attached updated > patch. > > On Wed, Sep 7, 2016 at 2:21 PM, Sergey Volk wrote: > > > Actual all

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Nicolas George
Le tridi 23 fructidor, an CCXXIV, Paul B Mahol a écrit : > You feed to nvenc only rgb? what testsrc only supports. Use testsrc2. That would be more efficient, but with testsrc, lavfi inserts the scale filter to convert into the requested format anyway. Regards, -- Nicolas George signature.a

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
>> for fmt in yuv420p nv12 bgr0 rgb0; do >> ./ffmpeg -f lavfi -i "testsrc=size=1920x1080:duration=10:rate=30" >> -c:v h264_nvenc -global_quality 20 -pix_fmt "$fmt" -y out_"${fmt}".mkv >> done > > You feed to nvenc only rgb? what testsrc only supports. Use testsrc2. pix_fmt should make sure it

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Clément Bœsch
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7 inse

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Paul B Mahol
On 9/8/16, Timo Rothenpieler wrote: > Am 08.09.2016 um 02:29 schrieb Sven C. Dack: >> On 08/09/16 00:57, Hendrik Leppkes wrote: >>> The image copying code was refactored in an earlier patch to be >>> generic and not rely on hard-coding format info, hence the second part >>> is not needed anymore.

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Sven C. Dack
On 08/09/16 08:46, Timo Rothenpieler wrote: Am 08.09.2016 um 02:29 schrieb Sven C. Dack: On 08/09/16 00:57, Hendrik Leppkes wrote: The image copying code was refactored in an earlier patch to be generic and not rely on hard-coding format info, hence the second part is not needed anymore. This

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
From: Aman Gupta --- libavcodec/videotoolboxenc.c | 76 ++-- 1 file changed, 67 insertions(+), 9 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 4345ca3..859dde9 100644 --- a/libavcodec/videotoolboxenc.c +++ b/l

[FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-08 Thread qw
Hi, I have made one simple transcoding program by using ffmpeg lib, where several av transcoding tasks are done in one process and each task corresponds to one thread. But some ffmpeg function will report error, and sometimes the multi-thread version of transcoding program will crash abnorma

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It does

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It does

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
> > Common attributes for AdaptationSet and Representation shall either be in > > one of the elements but not in both. > > > > in particular, the attribute frameRate is duplicated in the node > > Representation > > There is a bug report outstanding for this: > https://trac.ffmpeg.org/ticket/5087