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

2014-10-29 Thread Mika Raento
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

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

2014-10-29 Thread Mika Raento
segment muxer PIDs were fixed in https://github.com/FFmpeg/FFmpeg/commit/502fc3b3d4b36015562d19d74f27d0a4ff835c4e by me On 29 October 2014 10:17, Mika Raento mi...@iki.fi 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

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

2014-10-29 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at 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

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

2014-10-29 Thread Carl Eugen Hoyos
Clément Bœsch u at 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 at ref{pullup} filter. Well I don't mind much but then... how is pullup making any difference here

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

2014-10-29 Thread Carl Eugen Hoyos
Andrey Utkin andrey.krieger.utkin at 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

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

2014-10-29 Thread Christophe Gisquet
Hi, 2014-10-25 20:32 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: 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

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

2014-10-29 Thread wm4
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

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

2014-10-29 Thread Carl Eugen Hoyos
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

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

2014-10-29 Thread Carl Eugen Hoyos
wm4 nfxjfg at 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

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

2014-10-29 Thread Christophe Gisquet
Hi, 2014-10-25 13:29 GMT+02:00 Reimar Döffinger reimar.doeffin...@gmx.de: 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

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

2014-10-29 Thread Mika Raento
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

2014-10-29 Thread wm4
On Wed, 29 Oct 2014 09:35:32 + (UTC) Carl Eugen Hoyos ceho...@ag.or.at wrote: wm4 nfxjfg at 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

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

2014-10-29 Thread Michael Niedermayer
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 function

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

2014-10-29 Thread Thomas Mundt
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

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

2014-10-29 Thread Giorgio Vazzana
2014-10-21 12:26 GMT+02:00 Stefano Sabatini stefa...@gmail.com: 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

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

2014-10-29 Thread Michael Niedermayer
On Wed, Oct 29, 2014 at 03:20:02PM +0100, Giorgio Vazzana wrote: 2014-10-21 12:26 GMT+02:00 Stefano Sabatini stefa...@gmail.com: 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_

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

2014-10-29 Thread tomas . hardin
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

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

2014-10-29 Thread Calvin Walton
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 data

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

2014-10-29 Thread Calvin Walton
On Sat, 2014-10-25 at 12:22 +, Carl Eugen Hoyos wrote: Clément Bœsch u at 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 at ref{pullup} filter. Well I

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

2014-10-29 Thread Carl Eugen Hoyos
wm4 nfxjfg at 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 apparently

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

2014-10-29 Thread Carl Eugen Hoyos
Calvin Walton calvin.walton at 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

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

2014-10-29 Thread James Almer
On 29/10/14 11:20 AM, Giorgio Vazzana wrote: 2014-10-21 12:26 GMT+02:00 Stefano Sabatini stefa...@gmail.com: 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

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

2014-10-29 Thread Nicholas Robbins
On Wednesday, October 29, 2014 12:25 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Calvin Walton calvin.walton at 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

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

2014-10-29 Thread Carl Eugen Hoyos
wm4 nfxjfg at 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

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

2014-10-29 Thread Carl Eugen Hoyos
Nicholas Robbins nickrobbins-at-yahoo.com at 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 -

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

2014-10-29 Thread wm4
On Wed, 29 Oct 2014 17:30:47 + (UTC) Carl Eugen Hoyos ceho...@ag.or.at wrote: wm4 nfxjfg at 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

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

2014-10-29 Thread Mika Raento
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 mi...@iki.fi wrote: Created a feature request to support columnboxing input that changes aspect-ratio mid-stream.

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

2014-10-29 Thread Michael Niedermayer
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, Thomas Looks

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

2014-10-29 Thread Thomas Mundt
Thanks ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2014-10-29 Thread Thomas Mundt
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

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

2014-10-29 Thread Michael Niedermayer
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 binti...@gmail.com 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

2014-10-29 Thread Michael Niedermayer
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 2/3] ffplay: fix indentation after last commit

2014-10-29 Thread Marton Balint
Signed-off-by: Marton Balint c...@passwd.hu --- 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 @@

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

2014-10-29 Thread Marton Balint
The generic decoder properly flushes the codecs with one null packet as well. Signed-off-by: Marton Balint c...@passwd.hu --- 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 +++

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

2014-10-29 Thread Michael Niedermayer
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

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

2014-10-29 Thread Michael Niedermayer
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 http://www.google.com/url?q=http%3A%2F%2Fgit.videolan.org%2F%3Fp%3Dffmpeg.gitsa=Dsntz=1usg=AFQjCNEA5QH18TtMxcLhGx4b04pMUwSgYA], there is

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

2014-10-29 Thread Jan Ehrhardt
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

2014-10-29 Thread Bryan Huh
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 michae...@gmx.at wrote: On Tue, Oct 28, 2014 at 11:05:54PM +, Bryan

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

2014-10-29 Thread Michael Niedermayer
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, Michael