Re: [FFmpeg-devel] Experiences in using ffmpeg to transcode broadcast video

Hi Thanks for the feedback. I'll try to capture at least some of these in tickets, and reply on this thread as I go along. Async not working with damaged audio input is https://trac.ffmpeg.org/ticket/2693 (existing ticket) Sub2video not working with negative timestamps is https://trac.ffmpeg.or

Re: [FFmpeg-devel] Experiences in using ffmpeg to transcode broadcast video

segment muxer PIDs were fixed in https://github.com/FFmpeg/FFmpeg/commit/502fc3b3d4b36015562d19d74f27d0a4ff835c4e by me On 29 October 2014 10:17, Mika Raento wrote: > Hi > > Thanks for the feedback. > > I'll try to capture at least some of these in tickets, and reply on this > thread as I go alo

Re: [FFmpeg-devel] [RFC]Print a warning if a subtitle demuxer changes utf16 to utf8.

Nicolas George nsup.org> writes: > Le septidi 7 brumaire, an CCXXIII, Carl Eugen Hoyos a écrit : > > The user should be told by libavformat if it does something > > unexpected. > > Related to ticket #4059. > > I do not object to this quick fix The warning was merged by Michael. Thank you, Car

Re: [FFmpeg-devel] [RFC]Do not set the lame quality if the user didn't set it

Michael Niedermayer gmx.at> writes: > > Attached patch makes FFmpeg mp3 output more similar to lame's. > > It appears to me that 5 is not the default if vbr is used. > LGTM Merged by you. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-de

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

Clément Bœsch pkh.me> writes: > > +The filter only works for constant frame rate input. If your input > > +has mixed telecined and progressive content with changing framerate, > > +try the ref{pullup} filter. > > Well I don't mind much but then... how is pullup making > any difference here ac

Re: [FFmpeg-devel] [PATCH] Use v4l2 input format automatically if filename starts with "/dev/video"

Andrey Utkin gmail.com> writes: > +if (av_strstart(p->filename, "/dev/video", NULL)) > +return AVPROBE_SCORE_MAX; I took the freedom to apply this patch with MAX - 1. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:/

Re: [FFmpeg-devel] [PATCH 4/6] dv: fix weight table for 2x4x8 transform

Hi, 2014-10-25 20:32 GMT+02:00 Christophe Gisquet : > What I meant is I would need someone to: > 1) provide a command line to swap fields to produce an artificially > interlaced image > 2) confirm that -flags ildct sets CODEC_FLAG_INTERLACED_DCT and does > what is needed Unfortunately, I haven't

[FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

Other applications don't have this problem. The warning is useful for ffmpeg.c only, so it should be in ffmpeg.c. Note that the same problem exists for e.g. Matroska, which always uses UTF-8 for subtitles. If anyone wants to solve this properly, something somewhere else should check whether the pa

[FFmpeg-devel] [PATCH]Improve the fieldmatch documentation about mixed telecined content

Hi! Attached patch is meant to improve the warning that mixed telecined and progressive content is not handled well by fieldmatch and decimate. I believe this paragraph is more useful in the fieldmatch section of the documentation, if it should be moved to the decimate section, please say so.

Re: [FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

wm4 googlemail.com> writes: > For now, this message is annoying and useless for > non-ffmpeg.c users, and as I understand, libav* is > supposed to serve these users too. Why don't you reduce the log level instead? > This partially reverts commit 19a6431e And why a partial revert? Carl Eugen

Re: [FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

On Wed, 29 Oct 2014 09:16:51 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > For now, this message is annoying and useless for > > non-ffmpeg.c users, and as I understand, libav* is > > supposed to serve these users too. > > Why don't you reduce the log level instead?

Re: [FFmpeg-devel] [PATCH]Improve the fieldmatch documentation about mixed telecined content

Carl Eugen Hoyos ag.or.at> writes: > Attached patch is meant to improve the warning that mixed > telecined and progressive content is not handled well by > fieldmatch and decimate. Ok'ed by Clément on irc and pushed. Thank you, Carl Eugen ___ ffmpe

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: AVC Intra support

On 2014-10-28 23:38, Thomas Mundt wrote: Carl Eugen, I changed the indentations and attached a unified diff. Since I´m on windows I used turtoise git. I hope that´s okay. Regards, Thomas Looks good to me /Tomas ___ ffmpeg-devel mailing list ffmpeg-d

Re: [FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

wm4 googlemail.com> writes: > > > For now, this message is annoying and useless for > > > non-ffmpeg.c users, and as I understand, libav* is > > > supposed to serve these users too. > > > > Why don't you reduce the log level instead? > > For what purpose? I was under the impression that a lo

Re: [FFmpeg-devel] Problem uploading results to f...@fate.ffmpeg.org

On Wed, Oct 29, 2014 at 03:53:04AM +0100, Ngassa Finjap wrote: > Hello, > > I have ran the FATE test suite on my Fedora host using this: ( make V=2 -j2 > SAMPLES=/var/fate/samples THREADS=2 fate) and I can't seem to authenticate > and upload the results. I had previously sent a key to for authent

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: AVC Intra support

On Wed, Oct 29, 2014 at 10:27:25AM +0100, tomas.har...@codemill.se wrote: > On 2014-10-28 23:38, Thomas Mundt wrote: > >Carl Eugen, I changed the indentations and attached a unified diff. > >Since I´m on windows I used turtoise git. I hope that´s okay. > > > >Regards, > >Thomas > > Looks good to m

Re: [FFmpeg-devel] [PATCH 0/6] dv: of inverse weight tables

Hi, 2014-10-25 13:29 GMT+02:00 Reimar Döffinger : > Maybe as a patch that puts it into tools? > I don't know if that's the right place, but I would kind of prefer to > have these kind of things around. > Probably overkill, but we have libavcodec/tableprint.h that has helpers > to generate files wi

Re: [FFmpeg-devel] Experiences in using ffmpeg to transcode broadcast video

Tickets for sub2video and async not working with non-monotonic input, using -copyts: https://trac.ffmpeg.org/ticket/4062 https://trac.ffmpeg.org/ticket/4064 Hm. Lots of my issues seem to relate to using -copyts. I had trouble keeping all video and all audio if there were periods where one or the

Re: [FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

On Wed, 29 Oct 2014 09:35:32 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > > > For now, this message is annoying and useless for > > > > non-ffmpeg.c users, and as I understand, libav* is > > > > supposed to serve these users too. > > > > > > Why don't you reduce th

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libutvideodec: Support YUV422P10

On Wed, Oct 29, 2014 at 06:54:36AM +0100, Jan Ehrhardt wrote: > Michael Niedermayer in gmane.comp.video.ffmpeg.devel (Sat, 25 Oct 2014 > 00:14:14 +0200): > >patch applied > > FFmpeg does not (cross)compile anymore on Ubuntu (GCC 4.9.1) after this > patch: > > libavcodec/libutvideodec.cpp: In func

Re: [FFmpeg-devel] Experiences in using ffmpeg to transcode broadcast video

Created a feature request to support columnboxing input that changes aspect-ratio mid-stream. https://trac.ffmpeg.org/ticket/4065 On 29 October 2014 13:49, Mika Raento wrote: > Tickets for sub2video and async not working with non-monotonic input, > using -copyts: > > https://trac.ffmpeg.org/tic

[FFmpeg-devel] [Patch]support more AVC Intra formats without SPS/PPS header

Hi, this patch will add support for AVC Intra 50 720p and 1080p without SPS/PPS header in mxf and mov demuxers. I got the SPS/PPS tables from libbmx. Regards, Thomas avci50.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: don't use avpriv_ prefix for internal functions

2014-10-21 12:26 GMT+02:00 Stefano Sabatini : > On date Tuesday 2014-10-21 03:56:06 -0300, James Almer encoded: >> No need to keep the old symbols around until a major bump since lavd >> functions >> with the avpriv_ prefix were never exposed. >> >> Signed-off-by: James Almer > > LGTM. Ping. Any

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: don't use avpriv_ prefix for internal functions

On Wed, Oct 29, 2014 at 03:20:02PM +0100, Giorgio Vazzana wrote: > 2014-10-21 12:26 GMT+02:00 Stefano Sabatini : > > On date Tuesday 2014-10-21 03:56:06 -0300, James Almer encoded: > >> No need to keep the old symbols around until a major bump since lavd > >> functions > >> with the avpriv_ prefix

Re: [FFmpeg-devel] [Patch]support more AVC Intra formats without SPS/PPS header

On 2014-10-29 14:04, Thomas Mundt wrote: Hi, this patch will add support for AVC Intra 50 720p and 1080p without SPS/PPS header in mxf and mov demuxers. I got the SPS/PPS tables from libbmx. Regards, Thomas Looks OK to me. /Tomas ___ ffmpeg-devel ma

Re: [FFmpeg-devel] FFMPEG : Redirecting Matroska muxed data to socket

On Wed, 2014-10-29 at 11:26 +0530, Parth Shah wrote: > Hi all, > > I am using FFMPEG library to mux H.264 and AAC frames to Matroska > (.mkv) > file. I can do that both using command line and C program. > > Now, instead of writing the muxed matroska data into file I want to > write these muxed d

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

On Sat, 2014-10-25 at 12:22 +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > > +The filter only works for constant frame rate input. If your > > > input > > > +has mixed telecined and progressive content with changing > > > framerate, > > > +try the ref{pullup} filter. > >

Re: [FFmpeg-devel] [PATCH] avformat: remove messages that are useful for ffmpeg.c only

wm4 googlemail.com> writes: > > > > Why don't you reduce the log level instead? > > > > > > For what purpose? > > > > I was under the impression that a lot of messages > > are currently output by the libraries and that you > > are against seeing one more with warning level. > > But I am appar

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

Calvin Walton kepstin.ca> writes: > For content that was in mpeg2 with field flags set > appropriate for display on an interlaced TV This is unrelated to this issue: FFmpeg simply ignores the flag and interprets the input as progressive. > which basically accounts for all DVD content I live

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: don't use avpriv_ prefix for internal functions

On 29/10/14 11:20 AM, Giorgio Vazzana wrote: > 2014-10-21 12:26 GMT+02:00 Stefano Sabatini : >> On date Tuesday 2014-10-21 03:56:06 -0300, James Almer encoded: >>> No need to keep the old symbols around until a major bump since lavd >>> functions >>> with the avpriv_ prefix were never exposed. >>>

[FFmpeg-devel] [PATCH] avformat/subtitles: reduce log level of UTF-16 warning

Applications can use this to silence the message. --- As suggested. Not a great solution, but does the job to me. --- libavformat/subtitles.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c index 7f4bdef..67624fc 100644 --- a/li

Re: [FFmpeg-devel] [PATCH] lavu/atomic: add support for the new memory model aware gcc built-ins

On 28/10/14 11:16 PM, Michael Niedermayer wrote: > On Mon, Oct 27, 2014 at 11:25:01PM -0300, James Almer wrote: >> __sync built-ins are considered legacy and will be deprecated. >> These new memory model aware built-ins have been available since GCC 4.7.0 >> >> Use them by default when available ex

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

> On Wednesday, October 29, 2014 12:25 PM, Carl Eugen Hoyos > wrote: > > Calvin Walton kepstin.ca> writes: > >> For content that was in mpeg2 with field flags set >> appropriate for display on an interlaced TV > > This is unrelated to this issue: > FFmpeg simply ignores the flag and inter

Re: [FFmpeg-devel] [PATCH] avformat/subtitles: reduce log level of UTF-16 warning

wm4 googlemail.com> writes: > if (s && (r->type == FF_UTF16LE || r->type == FF_UTF16BE)) > -av_log(s, AV_LOG_WARNING, > +av_log(s, AV_LOG_INFO, Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpe

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

Nicholas Robbins ffmpeg.org> writes: > Not now, but later, I would be interested in helping to > adapt decimate to do what we want. A detectfps filter > might be useful too, and would be a subset of the work. I believe if decimate just detects 23.9 -> 29.97 and 29.97 -> 23.9 it will fix nearl

Re: [FFmpeg-devel] [PATCH]Mention in the documentation that fieldmatch needs cfr input

> On Wednesday, October 29, 2014 1:33 PM, Carl Eugen Hoyos > wrote: > > Nicholas Robbins ffmpeg.org> writes: > >> Not now, but later, I would be interested in helping to >> adapt decimate to do what we want. A detectfps filter >> might be useful too, and would be a subset of the work. >

Re: [FFmpeg-devel] [PATCH] avformat/subtitles: reduce log level of UTF-16 warning

On Wed, 29 Oct 2014 17:30:47 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > if (s && (r->type == FF_UTF16LE || r->type == FF_UTF16BE)) > > -av_log(s, AV_LOG_WARNING, > > +av_log(s, AV_LOG_INFO, > > Patch applied. > > Thank you, Carl Eugen Thanks.

Re: [FFmpeg-devel] Experiences in using ffmpeg to transcode broadcast video

Created a feature request to automatically scale subtitles for HD video https://trac.ffmpeg.org/ticket/4066 On 29 October 2014 14:25, Mika Raento wrote: > Created a feature request to support columnboxing input that changes > aspect-ratio mid-stream. > > https://trac.ffmpeg.org/ticket/4065 > >

Re: [FFmpeg-devel] [Patch]support more AVC Intra formats without SPS/PPS header

On Wed, Oct 29, 2014 at 04:35:34PM +0100, tomas.har...@codemill.se wrote: > On 2014-10-29 14:04, Thomas Mundt wrote: > >Hi, this patch will add support for AVC Intra 50 720p and 1080p > >without SPS/PPS header in mxf and mov demuxers. I got the SPS/PPS > >tables from libbmx. > > > >Regards, > >Thom

Re: [FFmpeg-devel] [Patch]support more AVC Intra formats without SPS/PPS header

Thanks ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] mxfenc: fix indentation after last commit

Should have done this as second part of my AVCI Patch. Sorry! Regards, Thomas indent.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] "OPW Qualification Task: Enable daemon mode for FFserver"

>From 0fb7dcf1f126bd137e2b2025c5cd6cff4af65801 Mon Sep 17 00:00:00 2001 From: Binathi Bingi Date: Thu, 30 Oct 2014 01:14:08 +0530 Subject: [PATCH] ffserver: enable back deamon mode --- ffserver.c| 34 ++ ffserver_config.c | 2 -- 2 files changed, 30 inser

Re: [FFmpeg-devel] "OPW Qualification Task: Enable daemon mode for FFserver"

On Thu, Oct 30, 2014 at 02:14:59AM +0530, Binathi Bingi wrote: > From 0fb7dcf1f126bd137e2b2025c5cd6cff4af65801 Mon Sep 17 00:00:00 2001 > From: Binathi Bingi > Date: Thu, 30 Oct 2014 01:14:08 +0530 > Subject: [PATCH] ffserver: enable back deamon mode > > --- > ffserver.c| 34

Re: [FFmpeg-devel] mxfenc: fix indentation after last commit

On Wed, Oct 29, 2014 at 08:26:25PM +, Thomas Mundt wrote: > Should have done this as second part of my AVCI Patch. Sorry! > > Regards, > Thomas > mxfenc.c | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > bd730a65f3d562a29fd14a84d79e1104547a262b indent.patch >

[FFmpeg-devel] [PATCH 1/3] ffplay: implement separete audio decoder thread

Signed-off-by: Marton Balint --- ffplay.c | 265 --- 1 file changed, 153 insertions(+), 112 deletions(-) diff --git a/ffplay.c b/ffplay.c index a979164..24bcae2 100644 --- a/ffplay.c +++ b/ffplay.c @@ -121,7 +121,8 @@ typedef struct Pac

[FFmpeg-devel] [PATCH 2/3] ffplay: fix indentation after last commit

Signed-off-by: Marton Balint --- ffplay.c | 182 +++ 1 file changed, 89 insertions(+), 93 deletions(-) diff --git a/ffplay.c b/ffplay.c index 24bcae2..a3b34fd 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2424,105 +2424,101 @@ static int aud

[FFmpeg-devel] [PATCH 3/3] ffplay: only output null packet once on EOF

The generic decoder properly flushes the codecs with one null packet as well. Signed-off-by: Marton Balint --- ffplay.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/ffplay.c b/ffplay.c index a3b34fd..e04f622 100644 --- a/ffplay.c +++ b/ffplay.c @@

Re: [FFmpeg-devel] "OPW Qualification Task: Enable daemon mode for FFserver"

If you check the latest ffserver.conf file on GIT [ http://git.videolan.org/?p=ffmpeg.git ], there is NoDaemon option in it. So the patch is written as per latest versio

Re: [FFmpeg-devel] [PATCH] movenc: Always write the trun data_offset

On Tue, Oct 28, 2014 at 11:05:54PM +, Bryan Huh wrote: > The MOV_TRUN_DATA_OFFSET flag was always getting set, so data_offset is > being expected and read. The offset was computed correctly anyway during > fragment-flush. Alternatively, I could have selectively set the flag > only when base-dat

Re: [FFmpeg-devel] "OPW Qualification Task: Enable daemon mode for FFserver"

On Thu, Oct 30, 2014 at 07:49:10AM +0530, Binathi Bingi wrote: > If you check the latest ffserver.conf file on GIT [ > http://git.videolan.org/?p=ffmpeg.git > ], > there

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libutvideodec: Support YUV422P10

Michael Niedermayer in gmane.comp.video.ffmpeg.devel (Wed, 29 Oct 2014 13:20:58 +0100): >should be fixed Ack. Thanks. Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] movenc: Always write the trun data_offset

Yes. Even with his change to the if-statement, I am proposing the same change: to remove the if-statement. It would be nice to get this sanity-checked by Martin though. On Wed, Oct 29, 2014 at 7:24 PM, Michael Niedermayer wrote: > On Tue, Oct 28, 2014 at 11:05:54PM +, Bryan Huh wrote: > > T

Re: [FFmpeg-devel] [PATCH] movenc: Always write the trun data_offset

On Wed, Oct 29, 2014 at 08:49:11PM -0700, Bryan Huh wrote: > Yes. Even with his change to the if-statement, I am proposing the same > change: to remove the if-statement. > > It would be nice to get this sanity-checked by Martin though. ok, cc-ing martin > > On Wed, Oct 29, 2014 at 7:24 PM, Mic

[FFmpeg-devel] Encode2() function in libavcodec/avcodec.h

Hi, the attributes has been changed to (AVCodecContext *avctx, AVPacket *avpkt, const AVFrame *frame, int *got_packet_ptr); what is the exact change made in the function. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/li