Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: read project_name metadata

2015-01-05 Thread Clément Bœsch
On Mon, Jan 05, 2015 at 04:53:05PM -0800, Mark Reid wrote: Hi, MXF files generated by Media Composer or LibMXF can contain a project name property in the Preface. Lots of existing samples have them. http://samples.ffmpeg.org/MXF/issue2160/PW0805A0V01.4C5B5636.EFA330.mxf project_name:

[FFmpeg-devel] [PATCH 4/4] nvenc: Try and do interlaced encoding.

2015-01-05 Thread Philip Langdale
Doesn't work. Signed-off-by: Philip Langdale phil...@overt.org --- libavcodec/libnvenc.c | 7 +++ libavcodec/libnvenc.h | 3 +++ libavcodec/nvencoder.c | 6 -- 3 files changed, 14 insertions(+), 2 deletions(-) diff --git a/libavcodec/libnvenc.c b/libavcodec/libnvenc.c index

[FFmpeg-devel] [PATCH 2/4] Set bdirect mode

2015-01-05 Thread Philip Langdale
Signed-off-by: Philip Langdale phil...@overt.org --- libavcodec/libnvenc.c | 17 + 1 file changed, 17 insertions(+) diff --git a/libavcodec/libnvenc.c b/libavcodec/libnvenc.c index 6ba150e..3c64234 100644 --- a/libavcodec/libnvenc.c +++ b/libavcodec/libnvenc.c @@ -208,6 +208,23

[FFmpeg-devel] [PATCH 3/4] Implement b frame support.

2015-01-05 Thread Philip Langdale
To support b frames, we need to implement a queue of buffers, so that frames can be held, pending their future reference frames. The nvenc docs say that we need (num b frames) + 4 buffers, and the maximum number of b frames is 16, so we need 20 buffers. While we could allocate them dynamically,

Re: [FFmpeg-devel] [PATCH] avfilter/vf_idet: Add analyze_interlaced_flag mode

2015-01-05 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 05/01/15 16:57, Michael Niedermayer wrote: On Mon, Jan 05, 2015 at 01:00:41PM +, tim nicholson wrote: On 01/01/15 01:56, Michael Niedermayer wrote: This should allow us to insert idet before scale and let scale have interl=-1 as default

[FFmpeg-devel] [PATCH 0/4] Improvements the nvidia's nvenc implementation

2015-01-05 Thread Philip Langdale
As promised, here are my current stack of improvements over the base implementation. The interlacing one is probably pointless as the hardware doesn't seem to support it, but you'll know better than me. And hopefully you can explain the black magic in the DAR calculation. Philip Langdale (4):

[FFmpeg-devel] [PATCH 1/4] Correctly set Display Aspect Ratio in nvenc.

2015-01-05 Thread Philip Langdale
The encoder writes the display aspect ratio into the output frames, which a good player will respect, so let's make sure it's correct. The 1.02 scale factor is black magic, but produces correct results. I don't know what nvenc is doing. Signed-off-by: Philip Langdale phil...@overt.org ---

Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: support USLT tags

2015-01-05 Thread wm4
On Mon, 5 Jan 2015 21:10:31 +0100 Michael Niedermayer michae...@gmx.at wrote: On Mon, Jan 05, 2015 at 06:56:20PM +0100, wm4 wrote: I think this turned out pretty terrible. There's no good way to add new custom tags that write to AVFormatContext-metadata. --- libavformat/id3v2.c | 50

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2015-01-05 Thread Anshul Maheshwari
On Tue, Jan 6, 2015 at 5:47 AM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Jan 05, 2015 at 06:20:15PM +0530, Anshul wrote: On 01/04/2015 10:17 PM, Michael Niedermayer wrote: the table is constant and does not change, theres no need to have a copy of it in each context or to make

Re: [FFmpeg-devel] [PATCH] libavfilter/vf_blend: do not unconditionally compile both filters

2015-01-05 Thread Stefano Sabatini
On date Sunday 2015-01-04 12:31:30 +, Paul B Mahol encoded: Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_blend.c | 32 1 file changed, 20 insertions(+), 12 deletions(-) diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg mention both ffpreset/avpreset in documentation, remove superfluous example

2015-01-05 Thread Stefano Sabatini
On date Friday 2015-01-02 11:13:29 +0100, Werner Robitza encoded: [...] (By the way, should I use in-reply-to or do you want separate threads for new patches? The developer doc doesn't say.) I tend to prefer to have a single thread, so I can follow the evolution and discussion related to a

Re: [FFmpeg-devel] [PATCH] lavfi: Port mp=pp7 to libavfilter

2015-01-05 Thread Stefano Sabatini
On date Monday 2014-12-29 10:20:26 +0100, Stefano Sabatini encoded: On date Sunday 2014-12-28 19:45:18 +0530, arwa arif encoded: [...] From ebf5f36d9e9fc80264bb605dfee5c09cbf1f10ef Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 25 Dec 2014 09:50:24 +0530

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2015-01-05 Thread Anshul
On 01/04/2015 10:17 PM, Michael Niedermayer wrote: the table is constant and does not change, theres no need to have a copy of it in each context or to make it every time decode is called a simple static uint8_t parity_table[256]; or even static const uint8_t parity_table[256] = {...} done

Re: [FFmpeg-devel] [PATCH] avfilter/vf_idet: Add analyze_interlaced_flag mode

2015-01-05 Thread tim nicholson
On 01/01/15 01:56, Michael Niedermayer wrote: This should allow us to insert idet before scale and let scale have interl=-1 as default in that case Good thinking. How would interl=-1 then get set as default for auto inserted filters. Or did I miss something? Signed-off-by: Michael

Re: [FFmpeg-devel] Main git server issues

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 02:04:47AM +0100, Michael Niedermayer wrote: Hi all as you probably already noticed videolans git server is not working since yesterday and as they generously provide also our main git, ffmpeg git is neither. I do hope the server will be alive again soon/?tomorrow?

[FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/Makefile | 2 + libavfilter/allfilters.c | 2 + libavfilter/f_deinterleave.c | 143 +++ 3 files changed, 147 insertions(+) create mode 100644 libavfilter/f_deinterleave.c diff

Re: [FFmpeg-devel] [PATCH 1/2] libavfilter/unsharp opencl optimization

2015-01-05 Thread Titov, Alexey
Hi Wei, I will double check using yuv. What opencl platform/device you are running it on? Also, can you point me to /testfile/blueangles.m4v so I can reproduce correct and not correct outputs? Thanks! 2015-01-05 14:50 GMT+08:00 Titov, Alexey alexey.ti...@amd.com: Hi Wei, This is the first

Re: [FFmpeg-devel] mpegvideo : interlaced_frame with picture_structure == PICT_FRAME

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 03:17:32PM -0500, Vanista Herion wrote: Some MPEG 2 video will give bad interlacing information in AVFrame. If the parameters are used by a deinterlace filter such as yadif, frames will be processed needlessly and impair the quality. yes and i hope this will get

Re: [FFmpeg-devel] mpegvideo : interlaced_frame with picture_structure == PICT_FRAME

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 03:17:32PM -0500, Vanista Herion wrote: Some MPEG 2 video will give bad interlacing information in AVFrame. If the parameters are used by a deinterlace filter such as yadif, frames will be processed needlessly and impair the quality. By reviewing the code in

Re: [FFmpeg-devel] mpegvideo : interlaced_frame with picture_structure == PICT_FRAME

2015-01-05 Thread Hendrik Leppkes
On Mon, Jan 5, 2015 at 9:17 PM, Vanista Herion vani...@gmail.com wrote: Some MPEG 2 video will give bad interlacing information in AVFrame. If the parameters are used by a deinterlace filter such as yadif, frames will be processed needlessly and impair the quality. By reviewing the code in

[FFmpeg-devel] [PATCH] libavformat/mxfdec.c: read project_name metadata

2015-01-05 Thread Mark Reid
Hi, MXF files generated by Media Composer or LibMXF can contain a project name property in the Preface. Lots of existing samples have them. http://samples.ffmpeg.org/MXF/issue2160/PW0805A0V01.4C5B5636.EFA330.mxf project_name: DNX145 PW Test

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 06:20:15PM +0530, Anshul wrote: On 01/04/2015 10:17 PM, Michael Niedermayer wrote: the table is constant and does not change, theres no need to have a copy of it in each context or to make it every time decode is called a simple static uint8_t parity_table[256];

Re: [FFmpeg-devel] [PATCH 1/2] libavfilter/unsharp opencl optimization

2015-01-05 Thread Wei Gao
2015-01-06 0:47 GMT+08:00 Titov, Alexey alexey.ti...@amd.com: Hi Wei, I will double check using yuv. What opencl platform/device you are running it on? Also, can you point me to /testfile/blueangles.m4v so I can reproduce correct and not correct outputs? Thanks! Hi Number of platforms:

Re: [FFmpeg-devel] [PATCH] avfilter/vf_idet: Add analyze_interlaced_flag mode

2015-01-05 Thread Michael Niedermayer
On Thu, Jan 01, 2015 at 02:56:41AM +0100, Michael Niedermayer wrote: This should allow us to insert idet before scale and let scale have interl=-1 as default in that case Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/filters.texi |7 +++ libavfilter/vf_idet.c |

[FFmpeg-devel] [PATCH] x86/swr: fix pack_8ch functions on compilers without aligned stack

2015-01-05 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- I don't have MSVC or ICL 10.x, so i only tested this with gcc after forcing HAVE_ALIGNED_STACK to 0 in config.asm libswresample/x86/audio_convert.asm| 39 -- libswresample/x86/audio_convert_init.c | 4 ++-- 2

[FFmpeg-devel] [PATCH 1/2] avformat/id3v1: strip trailing whitespace

2015-01-05 Thread wm4
ID3v1 fields have a fixed size, and they are padded either with zeros, or with spaces. Handle the latter case, instead of putting strings with trailing spaces into the AVDictionary. --- libavformat/id3v1.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 2/2] avformat/id3v2: support USLT tags

2015-01-05 Thread wm4
I think this turned out pretty terrible. There's no good way to add new custom tags that write to AVFormatContext-metadata. --- libavformat/id3v2.c | 50 ++ 1 file changed, 50 insertions(+) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c

Re: [FFmpeg-devel] [PATCH 1/2] avformat/id3v1: strip trailing whitespace

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 06:56:19PM +0100, wm4 wrote: ID3v1 fields have a fixed size, and they are padded either with zeros, or with spaces. Handle the latter case, instead of putting strings with trailing spaces into the AVDictionary. --- libavformat/id3v1.c | 11 ++- 1 file

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-05 Thread Michael Niedermayer
On Mon, Jan 05, 2015 at 04:33:44PM +, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/Makefile | 2 + libavfilter/allfilters.c | 2 + libavfilter/f_deinterleave.c | 143 +++ 3 files changed, 147

Re: [FFmpeg-devel] [PATCH] x86/flacdsp: remove unneeded ifdeffery

2015-01-05 Thread James Almer
On 04/01/15 8:49 PM, Michael Niedermayer wrote: On Sat, Jan 03, 2015 at 07:52:17PM -0300, James Almer wrote: On 29/12/14 5:05 PM, James Almer wrote: x86inc can translate r*m into a register or stack on its own Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/flacdsp.asm | 18

[FFmpeg-devel] [PATCH] libavformat/mxfdec.c: export the full UMID as metadata

2015-01-05 Thread Mark Reid
Hi, This patch exports the full umid of packages as metadata. ffmpeg currently only exports the material number portion of the umid. The new format is ISO label-length-instance number-material number example: 060a2b340101010101010f00-13-00-53dc416b9a770251060e2b347f7f2a80 Some

[FFmpeg-devel] [PATCH] libavformat/mxfdec.c: export the full UMID as metadata

2015-01-05 Thread Mark Reid
--- libavformat/mxfdec.c | 39 --- 1 file changed, 32 insertions(+), 7 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 4715169..5ade23a 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -200,6 +200,7 @@ typedef struct

[FFmpeg-devel] mpegvideo : interlaced_frame with picture_structure == PICT_FRAME

2015-01-05 Thread Vanista Herion
Some MPEG 2 video will give bad interlacing information in AVFrame. If the parameters are used by a deinterlace filter such as yadif, frames will be processed needlessly and impair the quality. By reviewing the code in mpegvideo.c which sets the interlaced_frame and top_field_first, I noticed