[FFmpeg-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-07-28 Thread Kieran Kunhya
--- doc/APIchanges|4 libavcodec/avcodec.h |5 - libavcodec/mpeg12dec.c| 21 - libavcodec/version.h |5 - libavfilter/vf_showinfo.c |3 +++ libavutil/frame.h | 12 libavutil/version.h |2

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
And as far as I can tell if you want to remux but with separate subtitle stream that would even mean that you have to re-encode the video for no good reason. You can just swap out the caption data. It's guaranteed to be CBR anyway. ___ ffmpeg-devel

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
Uh, no. The point is to get the CC data out you need to decode. Now that you have the video decoded, if you want to mux it you have to encode it again. Whether you decode the video or not is unrelated to your ability to replace the SEI and replace the captions. You'd still need to know the

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
into the decoders. The decoder approch would also not work for insertion of closed captions (for h.264 that would be x264, and I can't see that CC has a place there really). I'm uncertain enough that I would prefere to here the opinion here before submitting a patch. As far as I can see it

[FFmpeg-devel] [PATCH] h264: Move AFD to side data to match MPEG-2

2014-08-08 Thread Kieran Kunhya
--- libavcodec/h264.c | 11 +++ libavcodec/h264.h |6 ++ libavcodec/h264_sei.c |6 ++ 3 files changed, 23 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 8fa35c7..e77b633 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -883,6

[FFmpeg-devel] [PATCH] avcodec: Instruct users to align buffers to 32-byte.

2014-08-09 Thread Kieran Kunhya
The Opus decoder in particular uses optimised float_dsp functions that expect 32-byte alignment --- libavcodec/avcodec.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 93aad35..5cbc175 100644 --- a/libavcodec/avcodec.h +++

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-10 Thread Kieran Kunhya
On 10 August 2014 13:38, Michael Niedermayer michae...@gmx.at wrote: On Sat, Aug 09, 2014 at 06:26:19PM +0100, Kieran Kunhya wrote: [...] ... and was designed by a larger group instead of libswresample which was basically one person (and literally appeared in git out of nowhere). http

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
do you think me, the rest of the FFmpeg and Libav developers sitting together and people accusing each other of theft and lies. Would lead to improved relations ? You perhaps fail to realise since you've never done it before that there's vastly less of these long, stupid historical tales when

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
like i said, iam happy to resign, if that reunites the projects and resolves the problems how can my resignation not be enough, if as you say iam the blocking factor ? You impose conditions on your resignation. And wow, you'd rather resign than meet in person...

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
The social aspect should IMO not overlap over the technical one. Problems with the way people behave are social problems... In an ideal world yes, all the disagreements could be technical but in reality they are sadly not all technical. Kieran ___

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-16 Thread Kieran Kunhya
And do not forget the language barrier: when writing a mail, there is as much time as necessary to find words. IRL, the people struggling to find their words are at a clear disadvantage. We can rotate between different languages every ten minutes =p

Re: [FFmpeg-devel] [RFC] removing libmpcodecs for reuniting purposes.

2014-08-18 Thread Kieran Kunhya
On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote: ilpack - interlaced yuv420- yuv422 converter. Scale should be able to do that too. Scale doesn't have much (any?) knowledge of interlaced chroma. That said I could probably port this filter to lavfi because it would be quite

Re: [FFmpeg-devel] [RFC] removing libmpcodecs for reuniting purposes.

2014-08-18 Thread Kieran Kunhya
On 18 August 2014 14:37, Ivan Kalvachev ikalvac...@gmail.com wrote: On 8/18/14, Kieran Kunhya kier...@obe.tv wrote: On 18 August 2014 02:26, Ivan Kalvachev ikalvac...@gmail.com wrote: ilpack - interlaced yuv420- yuv422 converter. Scale should be able to do that too. Scale doesn't have much

Re: [FFmpeg-devel] [RFC] removing libmpcodecs for reuniting purposes.

2014-08-18 Thread Kieran Kunhya
How is it different? If you're interlacing-aware and call swscale_convert() twice (once for each field, double stride each, alternate offset for second call), isn't that the same? That would be true in the 422 domain, yes. In 420, the chroma planes are offset and this has to be taken into

Re: [FFmpeg-devel] [PATCH] x86/hevc_res_add: refactor ff_hevc_transform_add{16, 32}_8

2014-08-21 Thread Kieran Kunhya
It does that, but on older SSE2 cpus with not-so-good OOO execution grouping instructions like this might help reduce dependencies a bit. Are any older SSE2 CPUs actually capable of decoding reasonable HEVC? ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-29 Thread Kieran Kunhya
think discussing the Libav-FFmpeg split and ways to resolve it at VDD makes a lot of sense, quite litterally more than 90% of the developers wont be there. I also wont be there Lame excuse. You're the leader and the main reason why people want you there. also iam quite confident that if

[FFmpeg-devel] Patch: 42507534977e2108e397a381357ee2db209475f6

2014-09-02 Thread Kieran Kunhya
Patch not sent to ML so couldn't review: Why have you only changed the code to favour DESC_types and not other types? Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Patch: 42507534977e2108e397a381357ee2db209475f6

2014-09-02 Thread Kieran Kunhya
On 3 September 2014 01:40, Michael Niedermayer michae...@gmx.at wrote: On Wed, Sep 03, 2014 at 12:59:53AM +0100, Kieran Kunhya wrote: Patch not sent to ML so couldn't review: Why have you only changed the code to favour DESC_types and not other types? because thats what was needed to fix

Re: [FFmpeg-devel] Patch: 42507534977e2108e397a381357ee2db209475f6

2014-09-03 Thread Kieran Kunhya
My question is not really about testcases etc but more why you decided to not include the other descriptor types. I guess I can send a patch to fix that. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

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

2014-09-27 Thread Kieran Kunhya
On 28 September 2014 00:49, Michael Niedermayer michae...@gmx.at wrote: This allows remuxing AVC intra into mov it does not work with libx264 encoded AVC Intra for unknown reason 1080i should work. 1080p needs some random magic numbers in the bitstream which are probably different for all the

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

2014-09-30 Thread Kieran Kunhya
Some of the DPP AVCI100 test samples were created using x264* so could be used for reference. Or I could knock up something if I can find my reference CL I used last time. www.obe.tv/about-us/obe-blog/item/1-oss-dpp-creation Remember to use the latest x264. Even then the values in the UMID

[FFmpeg-devel] [PATCH] Add support for Opus in MPEG-TS

2014-10-17 Thread Kieran Kunhya
--- libavcodec/opus.c| 11 +--- libavcodec/opus.h|9 +++ libavcodec/opus_parser.c | 139 +- libavformat/mpegts.c | 54 +- 4 files changed, 191 insertions(+), 22 deletions(-) diff --git a/libavcodec/opus.c

Re: [FFmpeg-devel] [PATCH] Add support for Opus in MPEG-TS

2014-10-27 Thread Kieran Kunhya
On 26 October 2014 21:32, Michael Niedermayer michae...@gmx.at wrote: On Sat, Oct 18, 2014 at 12:25:16AM +0100, Kieran Kunhya wrote: --- libavcodec/opus.c| 11 +--- libavcodec/opus.h|9 +++ libavcodec/opus_parser.c | 139

Re: [FFmpeg-devel] [PATCH] avformat/udp: UDP-Lite (RFC 3828) support added

2014-11-03 Thread Kieran Kunhya
On 3 November 2014 14:35, Michael Niedermayer michae...@gmx.at wrote: On Mon, Nov 03, 2014 at 09:35:16AM +0100, Thomas Volkert wrote: From: Thomas Volkert tho...@homer-conferencing.com --- Changelog| 1 + configure| 3 +++ libavformat/allformats.c | 1 +

[FFmpeg-devel] [PATCH] swscale: Enable yuv2planeX_8 assembly that had been disabled for some reason

2014-11-08 Thread Kieran Kunhya
From 196dd0d9eb40ca18e3bba993f5198bd1bde536ce Mon Sep 17 00:00:00 2001 From: Kieran Kunhya kier...@ob-encoder.com Date: Sat, 8 Nov 2014 20:34:25 -0600 Subject: [PATCH] swscale: Enable yuv2planeX_8 assembly that had been disabled for some reason --- libswscale/x86/swscale.c |2 +- 1 file

[FFmpeg-devel] [PATCH] swscale: fix yuv2yuvX_8 assembly on x86

2014-11-09 Thread Kieran Kunhya
--- libswscale/x86/swscale.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libswscale/x86/swscale.c b/libswscale/x86/swscale.c index 8ce87b3..c9f3b1a 100644 --- a/libswscale/x86/swscale.c +++ b/libswscale/x86/swscale.c @@ -430,7 +430,7 @@ switch(c-dstBpc){ \ case 16:

Re: [FFmpeg-devel] [PATCH] vf_interlace: Add SIMD for lowpass filter

2014-11-10 Thread Kieran Kunhya
Can't pavgw be used here? I don't think so? Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] v210dec: Fix width calculation

2014-11-22 Thread Kieran Kunhya
--- libavcodec/v210dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c index ae03952..978dffe 100644 --- a/libavcodec/v210dec.c +++ b/libavcodec/v210dec.c @@ -117,7 +117,7 @@ static int decode_frame(AVCodecContext *avctx, void

Re: [FFmpeg-devel] [PATCH] v210dec: Fix width calculation

2014-11-22 Thread Kieran Kunhya
On 22 November 2014 at 19:32, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sat, Nov 22, 2014 at 07:11:57PM +, Kieran Kunhya wrote: --- libavcodec/v210dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c index

[FFmpeg-devel] [PATCH] v210enc: Add x86 SIMD

2014-11-22 Thread Kieran Kunhya
--- libavcodec/v210enc.c| 78 ++--- libavcodec/v210enc.h| 31 libavcodec/x86/Makefile | 2 ++ 3 files changed, 88 insertions(+), 23 deletions(-) create mode 100644 libavcodec/v210enc.h diff --git a/libavcodec/v210enc.c

[FFmpeg-devel] [PATCH] v210enc: Add x86 SIMD

2014-11-22 Thread Kieran Kunhya
@@ +;** +;* V210 SIMD pack +;* Copyright (c) 2014 Kieran Kunhya kier...@obe.tv +;* +;* This file is part of Libav. +;* +;* Libav is free software; you can redistribute it and/or +;* modify it under the terms of the GNU Lesser General Public +;* License as published by the Free

Re: [FFmpeg-devel] [libav-devel] [PATCH] v210enc: Add x86 SIMD

2014-11-23 Thread Kieran Kunhya
You may be able to use mova below as well, but I don't know if AVFrame-data and AVPacket-data are aligned here. It's probably worth a try. Won't work because 12 bytes at a time are processed. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] v210enc: Directly encode YUV422P to v210. About 6 times faster than existing (broken) swscale based path

2014-11-23 Thread Kieran Kunhya
--- libavcodec/v210enc.c | 151 +++--- libavcodec/v210enc.h | 4 +- libavcodec/x86/v210enc.asm| 104 - libavcodec/x86/v210enc_init.c | 13 +++- libavutil/x86/x86util.asm | 5 ++ 5 files changed, 218

[FFmpeg-devel] [PATCH] vf_interlace: get rid of useless loads

2014-11-25 Thread Kieran Kunhya
--- libavfilter/x86/vf_interlace.asm |2 -- 1 file changed, 2 deletions(-) diff --git a/libavfilter/x86/vf_interlace.asm b/libavfilter/x86/vf_interlace.asm index b8d8616..85811da 100644 --- a/libavfilter/x86/vf_interlace.asm +++ b/libavfilter/x86/vf_interlace.asm @@ -42,8 +42,6 @@ cglobal

[FFmpeg-devel] [PATCHv2] v210enc: Directly encode YUV422P to v210. About 6 times faster than existing (broken) swscale based path

2014-11-25 Thread Kieran Kunhya
--- libavcodec/v210enc.c | 151 +++--- libavcodec/v210enc.h | 4 +- libavcodec/x86/v210enc.asm| 102 +++- libavcodec/x86/v210enc_init.c | 13 +++- libavutil/x86/x86util.asm | 5 ++ 5 files changed, 217

[FFmpeg-devel] [PATCH] v210enc: Add SIMD optimised 8-bit and 10-bit encoders

2014-11-26 Thread Kieran Kunhya
@@ +;** +;* V210 SIMD pack +;* Copyright (c) 2014 Kieran Kunhya kier...@obe.tv +;* +;* This file is part of FFmpeg. +;* +;* FFmpeg is free software; you can redistribute it and/or +;* modify it under the terms of the GNU Lesser

Re: [FFmpeg-devel] [PATCH] v210enc: Add SIMD optimised 8-bit and 10-bit encoders

2014-11-26 Thread Kieran Kunhya
v210_enc_chroma_shuf1_8: db 0,-1,1,-1,2,-1,3,-1,8,-1,9,-1,10,-1,11,-1 -v210_enc_chroma_shuf2_8: db 4,-1,5,-1,6,-1,7,-1,12,-1,13,-1,14,-1,15,-1 +v210_enc_chroma_shuf2_8: db 3,-1,4,-1,5,-1,7,-1,11,-1,12,-1,13,-1,15,-1 v210_enc_chroma_mult_8: dw 4,16,64,0,64,4,16,0 Thanks Th

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tinterlace: Favor using standard timebases for the output

2014-12-01 Thread Kieran Kunhya
On 1 December 2014 at 20:04, Michael Niedermayer michae...@gmx.at wrote: Inspired by discussion with kierank Signed-off-by: Michael Niedermayer michae...@gmx.at Can you explain what this patch does and how it affects all the different options in tinterlace? Kieran

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tinterlace: Favor using standard timebases for the output

2014-12-01 Thread Kieran Kunhya
On 1 December 2014 at 20:26, Michael Niedermayer michae...@gmx.at wrote: On Mon, Dec 01, 2014 at 08:17:18PM +, Kieran Kunhya wrote: On 1 December 2014 at 20:04, Michael Niedermayer michae...@gmx.at wrote: Inspired by discussion with kierank Signed-off-by: Michael Niedermayer michae

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tinterlace: Favor using standard timebases for the output

2014-12-01 Thread Kieran Kunhya
This effectively limits interlacing to two framerates. What about pure 30i? What about some future (or past) framerate we didn't think of? Listing all possible framerate combinations is simply not maintainable. Erm, let's not propagate more crazy interlaced framerates. If you reeally

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tinterlace: Favor using standard timebases for the output

2014-12-01 Thread Kieran Kunhya
Sent from my mobile device On 1 Dec 2014 21:55, Michael Niedermayer michae...@gmx.at wrote: On Mon, Dec 01, 2014 at 08:54:16PM +, Vittorio Giovara wrote: On Mon, Dec 1, 2014 at 8:04 PM, Michael Niedermayer michae...@gmx.at wrote: Inspired by discussion with kierank Signed-off-by:

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

2014-12-04 Thread Kieran Kunhya
Furthermore, I do not see any ASS styling in the decoder, that would be a good test case. Captions shouldn't be ASS styled - they should be styled as per the specification (see VLC). Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] Parallelization of FFmpeg

2014-12-22 Thread Kieran Kunhya
On 22 December 2014 at 01:43, Dtison.net systemad...@dtison.net wrote: Hello Would there be any interest in additional work on parallelization of the library? It is mentioned the Changelog and in multithreading.txt. What part of FFmpeg are you interested in Parallelising? Kieran

[FFmpeg-devel] Frame threaded encoders

2014-12-22 Thread Kieran Kunhya
Hi, Does the CODEC_CAP_FRAME_THREADS API support frame threaded encoding? Regards, Kieran Kunhya ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [RFC][PATCH] Use correct chroma position in YUV420P interlaced conversions

2014-12-24 Thread Kieran Kunhya
I think this is right but would be useful to get it checked. Visually the output looks ok --- libavfilter/vf_scale.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c index 64b88c2..83a0666 100644 ---

[FFmpeg-devel] [PATCHv2] vf_scale: Use correct chroma position in YUV420P interlaced conversions

2014-12-24 Thread Kieran Kunhya
Fixed wrong chroma line use --- libavfilter/vf_scale.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c index 64b88c2..9189103 100644 --- a/libavfilter/vf_scale.c +++ b/libavfilter/vf_scale.c @@ -373,6 +373,13 @@

Re: [FFmpeg-devel] [PATCHv2] vf_scale: Use correct chroma position in YUV420P interlaced conversions

2014-12-24 Thread Kieran Kunhya
+ av_opt_set_int(*s, src_h_chr_pos, scale-in_h_chr_pos, 0); av_opt_set_int(*s, src_v_chr_pos, scale-in_v_chr_pos, 0); av_opt_set_int(*s, dst_h_chr_pos, scale-out_h_chr_pos, 0); @@ -520,8 +527,8 @@ static int filter_frame(AVFilterLink *link, AVFrame *in)

[FFmpeg-devel] [PATCHv3] vf_scale: Use correct chroma positions for YUV420P

2014-12-24 Thread Kieran Kunhya
Merry Christmas! --- libavfilter/vf_scale.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c index 64b88c2..f77884c 100644 --- a/libavfilter/vf_scale.c +++ b/libavfilter/vf_scale.c @@ -373,6 +373,17 @@ static int

[FFmpeg-devel] [PATCH] swscale: Pass through chroma positions in sws_getCachedContext

2014-12-26 Thread Kieran Kunhya
--- libswscale/utils.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libswscale/utils.c b/libswscale/utils.c index ab494ed..601e7bf 100644 --- a/libswscale/utils.c +++ b/libswscale/utils.c @@ -1960,6 +1960,8 @@ struct SwsContext *sws_getCachedContext(struct SwsContext

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-08 Thread Kieran Kunhya
It has timestamp inside the data stream, and for making a simple use case scte need -copyts and -vsync 0 param. Otherwise user can ignore the data as they always did. -Anshul Is FFmpeg aware of this PTS and how to place the data packet with respect to PES packets? How are you testing this

Re: [FFmpeg-devel] Enable stream copy of data

2015-01-14 Thread Kieran Kunhya
On 14 January 2015 at 09:41, Anshul anshul.ffm...@gmail.com wrote: Hi I have enabled demuxing and muxing path for datat stream How do you guarantee accuracy of the SCTE-35 stream without a timestamp? Note that ffmpeg isn't able to understand timestamps here because it should be using an

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 23:00, Nicolas George geo...@nsup.org wrote: L'octidi 28 nivôse, an CCXXIII, Kieran Kunhya a écrit : I don't make the standards and frankly whether you dislike them is your problem but they exist and need to work correctly. Instead you wish to break things based off

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 20:01, Philip Langdale phil...@overt.org wrote: There is a long sad story behind all this, but it's somewhat ambiguous as to whether DVD content should be treated as 720 pixels wide or 704 pixels, with 16 pixels cut off. If you decide is should be 704 pixels wide, you need

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 23:38, Nicolas George geo...@nsup.org wrote: L'octidi 28 nivôse, an CCXXIII, Kieran Kunhya a écrit : The behaviour of the Nvidia code I believe is correct. As far as I understand it corrects SAR for 720-width content to comply with BT601. That is just not true. Basic

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvenc: Handle non-square pixel aspect ratios

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 18:14, Nicolas George geo...@nsup.org wrote: Le septidi 27 nivôse, an CCXXIII, Philip Langdale a écrit : On Fri, 16 Jan 2015 22:17:56 +0100 Nicolas George geo...@nsup.org wrote: Ok. I did this test and it produces correct results - SAR 133:221 which yields the correct

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 20:42, Nicolas George geo...@nsup.org wrote: L'octidi 28 nivôse, an CCXXIII, Kieran Kunhya a écrit : BT601 makes this very clear. The active picture is 702 pixels. There are two fundamental flaws with your reasoning: First, BT601 only applies to a some kind of videos

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-17 Thread Kieran Kunhya
On 17 January 2015 at 23:46, Kieran Kunhya kier...@obe.tv wrote: On 17 January 2015 at 23:38, Nicolas George geo...@nsup.org wrote: L'octidi 28 nivôse, an CCXXIII, Kieran Kunhya a écrit : The behaviour of the Nvidia code I believe is correct. As far as I understand it corrects SAR for 720

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvenc: Handle non-square pixel aspect ratios

2015-01-17 Thread Kieran Kunhya
FFmpeg is correct, there is absolutely no doubt about it: This active pixels nonsense is only relevant for certain very specific media, definitely not when encoding testsrc and decoding the result to showinfo. Demuxers or high-level tools may know when they are dealing with that kind of

Re: [FFmpeg-devel] [PATCH] avcodec/h264: Partially decode and display single fields try #2

2015-01-21 Thread Kieran Kunhya
yes, its overall 3 lines shorter as well ive locally changed it to this: Thanks, much easier to understand. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-18 Thread Kieran Kunhya
On 18 January 2015 at 09:40, Nicolas George geo...@nsup.org wrote: Le nonidi 29 nivôse, an CCXXIII, Hendrik Leppkes a écrit : nvenc should behave the same as libx264, or any other video encoder, if this patch makes it do that, then it should be applied. If bt601 needs special handling not yet

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-18 Thread Kieran Kunhya
On 18 January 2015 at 12:22, Michael Niedermayer michae...@gmx.at wrote: On Sun, Jan 18, 2015 at 02:52:59AM +0100, Hendrik Leppkes wrote: On Sun, Jan 18, 2015 at 1:02 AM, Nicolas George geo...@nsup.org wrote: L'octidi 28 nivôse, an CCXXIII, Kieran Kunhya a écrit : Oops I misunderstood, you

Re: [FFmpeg-devel] [PATCH] avfilter: Port mp=softpulldown to lavfi

2015-01-29 Thread Kieran Kunhya
On 29 January 2015 at 09:20, Paul B Mahol one...@gmail.com wrote: On 1/29/15, Michael Niedermayer michae...@gmx.at wrote: On Wed, Jan 28, 2015 at 03:13:27PM +, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com --- Bit-exact with mp=softpulldown except first frame which is

Re: [FFmpeg-devel] [PATCH] avfilter: Port mp=softpulldown to lavfi

2015-01-29 Thread Kieran Kunhya
On 29 January 2015 at 09:57, Kieran Kunhya kier...@obe.tv wrote: On 29 January 2015 at 09:20, Paul B Mahol one...@gmail.com wrote: On 1/29/15, Michael Niedermayer michae...@gmx.at wrote: On Wed, Jan 28, 2015 at 03:13:27PM +, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com

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

2015-01-11 Thread Kieran Kunhya
no, there where some comments on IRC but IIRC they where along the lines of future improvments not objections/blocking Has any work been done on whether this implementation matches the reference vectors as I mentioned on IRC? Kieran ___ ffmpeg-devel

Re: [FFmpeg-devel] Enable stream copy of data

2015-02-09 Thread Kieran Kunhya
I am using bmd device to test this. No one came with more scte_35 video on trac, I thought people putting the bounty will give me some video. How are you testing with a bmd device? As far as I can tell it just works by chance. I have done some improvement lately now it work without copyts

Re: [FFmpeg-devel] [PATCH 2/2] doc/faq: explain DAR/SAR preserving.

2015-02-09 Thread Kieran Kunhya
+A lot of video codecs and formats can store the @emph{aspect ratio} of the +video: this is the ratio between the width and the height of either the full +image (DAR, display aspect ratio) or individual pixels (SAR, sample aspect +ratio). For example, CGA screens at resolution 640×350 had 4:3

Re: [FFmpeg-devel] DCA Decoder

2015-03-12 Thread Kieran Kunhya
On 11 March 2015 at 14:42, Marcus Johnson bumblebritche...@gmail.com wrote: I've been working on adding XLL for the last couple months, it's still not quite complete, basically I have to combine the Core and XLL samples before it's output, and I also have to finish the latter stages of decoding

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Kieran Kunhya
And you didn't answer the question I asked before: Why would only AC3 have range compression on by default? Why not any other codec? What is the sense in that?? Are you saying that laptop speakers are not able to play full range AC3 tracks, but that they *are* able to play full range TrueHD

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Kieran Kunhya
On 30 March 2015 at 11:24, madshi mad...@gmail.com wrote: 2015-03-30 12:10 GMT+02:00 Kieran Kunhya kier...@obe.tv: We should have DRC enabled in AAC and other codecs if it's considered a normative part of the spec. So basically you do what the spec says regardless of whether it makes sense

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-29 Thread Kieran Kunhya
On 29 March 2015 at 18:01, Wiebe Cazemier wi...@halfgaar.net wrote: Signed-off-by: Wiebe Cazemier wi...@halfgaar.net Why can't you just set this in your application? Not everyone is listening on a full setup where they can hear the dynamic range. Kieran

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Kieran Kunhya
Kieran, the material used in testing is CFR to the best of my knowedge the material does not start at pts 0, the code ignores the start time aka the first pts. Theres only one possibility to fix this and that is to use the first pts, no (de)telecine operation can maintain sync with another

Re: [FFmpeg-devel] [PATCH] lavfi: add inverse telecine filter

2015-03-23 Thread Kieran Kunhya
why does the filter not use the input pts? Michael, none of the PTS hacks will ever work - (inverse)telecine is a CFR operation. It inherently requires knowing the duration of the frame and in VFR mode you don't know it. Kieran ___ ffmpeg-devel

Re: [FFmpeg-devel] MPEG-4 audio/video file generation API

2015-02-22 Thread Kieran Kunhya
On 22 February 2015 at 23:08, Boris Jaulmes bjaul...@gmail.com wrote: Dear ffmpeg coderz, Yes the coderz got your email the first time. I am sure all the ffmpeg rockstar devs will be emailing you soon. Kieran ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH v4] libavcodec: Add FLAC API test

2015-04-20 Thread Kieran Kunhya
On 20 April 2015 at 01:03, Ludmila Glinskih lglins...@gmail.com wrote: Signed-off-by: Ludmila Glinskih lglins...@gmail.com --- libavcodec/Makefile| 1 + libavcodec/api-flac-test.c | 290 + I would put this in its own directory. I think

Re: [FFmpeg-devel] [PATCH v4] libavcodec: Add FLAC API test

2015-04-21 Thread Kieran Kunhya
On 21 April 2015 at 21:52, Ludmila Glinskih lglins...@gmail.com wrote: Sorry for the delay, thanks for comments! вт, 21 апр. 2015 г. в 1:40, Kieran Kunhya kier...@obe.tv: On 20 April 2015 at 01:03, Ludmila Glinskih lglins...@gmail.com wrote: Signed-off-by: Ludmila Glinskih lglins

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-14 Thread Kieran Kunhya
On 14 April 2015 at 14:01, Wiebe Cazemier wi...@halfgaar.net wrote: - Original Message - From: Wiebe Cazemier wi...@halfgaar.net To: Michael Niedermayer michae...@gmx.at Cc: FFmpeg development discussions and patches ffmpeg-devel@ffmpeg.org, madshi mad...@gmail.com Sent: Monday, 6

Re: [FFmpeg-devel] Remove codec specific metadata inside MDAT

2015-04-13 Thread Kieran Kunhya
Is there a way for me to remove the x264 specific metadata from the MDAT box of the mp4? Right now, my MDAT has the following as the first few bytes. Is this length accounted for as part of MDAT headers? This data is part of the video bitstream. If you want to hide your x264 settings please

Re: [FFmpeg-devel] Remove codec specific metadata inside MDAT

2015-04-13 Thread Kieran Kunhya
On 13 April 2015 at 12:49, Kamaldeep Tumkur kamaldeep.tum...@gmail.com wrote: I don't want to hide the x264 settings. If this is part of the bitstream, are all decoders going to understand it? Will it be used in some way? For decoders that do not understand it, will it cause issues or will it

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-09 Thread Kieran Kunhya
On 9 April 2015 at 12:17, Rodger Combs rodger.co...@gmail.com wrote: On Apr 9, 2015, at 06:08, wm4 nfx...@googlemail.com wrote: On Thu, 9 Apr 2015 00:02:50 +0200 Lukasz Marek lukasz.m.lu...@gmail.com wrote: W dniu środa, 8 kwietnia 2015 Gilles Chanteperdrix

Re: [FFmpeg-devel] [PATCH] avcodec/s302m: Check for non PCM Streams and provide a informative error message instead of noise

2015-06-04 Thread Kieran Kunhya
On 4 June 2015 at 15:57, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/s302m.c | 34 ++ 1 file changed, 34 insertions(+) This should be made optional - there are legitimate reasons such as

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avio: Extend API with avio_move() and avio_delete()

2015-06-21 Thread Kieran Kunhya
Suppose you're writing a video player with browsing capabilities for network protocols (like Kodi/XBMC). Now you can have file rename/delete functionality in it. Suppose you are writing a video player and need to change the screen resolution. Can we have that feature in libavutil too? Kieran

[FFmpeg-devel] [PATCH] avcodec: Add support for per-frame AFD output in h264

2015-06-21 Thread Kieran Kunhya
From 6c0c94f8581d9e76301b03f9f416972fc0265fb6 Mon Sep 17 00:00:00 2001 From: Kieran Kunhya kie...@kunhya.com Date: Sun, 21 Jun 2015 23:59:12 +0100 Subject: [PATCH] avcodec: Add support for per-frame AFD output in h264 --- libavcodec/h264.c | 11 +++ libavcodec/h264.h | 3

[FFmpeg-devel] [PATCHv2] avcodec: Add support for per-frame AFD output in h264

2015-06-26 Thread Kieran Kunhya
From: Kieran Kunhya kie...@kunhya.com --- libavcodec/h264.c | 10 ++ libavcodec/h264.h | 3 +++ libavcodec/h264_sei.c | 6 ++ 3 files changed, 19 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 9be317c..1cbd4cb 100644 --- a/libavcodec/h264.c +++ b

[FFmpeg-devel] [PATCH] avcodec: Add support for Closed Caption export in h264

2015-06-27 Thread Kieran Kunhya
--- libavcodec/h264.c | 10 ++ libavcodec/h264.h | 2 ++ libavcodec/h264_sei.c | 32 +++- 3 files changed, 43 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 1cbd4cb..d971c7b 100644 --- a/libavcodec/h264.c +++

Re: [FFmpeg-devel] [PATCH] vp9: don't retain NULL as segmentation_map

2015-06-18 Thread Kieran Kunhya
Thanks for this analysis. I've pushed the patch now. By the way, afl [1] is really a great tool for finding such problems. I can only recommend everyone to try it out yourself. Not to push this off-topic but can you let me know how you use afl with ffmpeg - I could never get it to work when I

Re: [FFmpeg-devel] FFmpeg/MPlayer/rtmpdump possibly searching for a new server and hosting

2015-07-03 Thread Kieran Kunhya
How much equipment or server resources do you need. I should be able to host this. Kieran On 3 July 2015 at 18:53, Michael Niedermayer michae...@gmx.at wrote: Hi all It is POSSIBLE that we need to move to different servers/hosting. We have been informed that the free hosting and servers we

[FFmpeg-devel] [PATCHv2] avcodec: Add support for Closed Caption export in h264

2015-06-27 Thread Kieran Kunhya
Fix missing line --- libavcodec/h264.c | 11 +++ libavcodec/h264.h | 2 ++ libavcodec/h264_sei.c | 32 +++- 3 files changed, 44 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 1cbd4cb..1569ec8 100644 ---

Re: [FFmpeg-devel] [PATCH] api-h264-test: build with another api test

2015-06-28 Thread Kieran Kunhya
So I have a more conceptual question, maybe Kieran can help answer this also. Why are these tests called api-$codec tests? I don't see anything flac/h264 specific in these tests so far, and they could basically be reused for any video (h264) or audio (flac) test (for audio: as long as there's

[FFmpeg-devel] [PATCHv3] avcodec: Add support for Closed Caption export in h264

2015-07-02 Thread Kieran Kunhya
--- libavcodec/h264.c | 11 +++ libavcodec/h264.h | 2 ++ libavcodec/h264_sei.c | 35 ++- 3 files changed, 47 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 1cbd4cb..1569ec8 100644 --- a/libavcodec/h264.c +++

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_testsrc: correct colors for smptebars

2015-08-17 Thread Kieran Kunhya
On 10 August 2015 at 17:38, Paul B Mahol one...@gmail.com wrote: On 8/9/15, Kieran Kunhya kier...@obe.tv wrote: On 9 August 2015 at 12:20, Paul B Mahol one...@gmail.com wrote: --- libavfilter/vsrc_testsrc.c | 36 +++- 1 file changed, 23 insertions(+), 13

[FFmpeg-devel] Message to the community

2015-08-01 Thread Kieran Kunhya
that there is a proper response from the community instead of bits and pieces on IRC, Hacker News, Reddit etc. Regards, Kieran Kunhya ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] news: submit reworded announcement

2015-08-01 Thread Kieran Kunhya
On 2 August 2015 at 00:02, Clément Bœsch u...@pkh.me wrote: On Sun, Aug 02, 2015 at 12:53:59AM +0200, wm4 wrote: [...] This is a bit too long and redundant with the new paragraphs added. I don't understand why you felt the need to redo this at all (including reverting the already pushed

Re: [FFmpeg-devel] [PATCH] news: submit reworded announcement

2015-08-01 Thread Kieran Kunhya
On 2 August 2015 at 00:11, Clément Bœsch u...@pkh.me wrote: From: Kieran Kunhya kier...@ob-encoder.com Signed-off-by: Lydia Pintscher ly...@kde.org --- src/index | 30 ++ 1 file changed, 30 insertions(+) Apologies for my previous outburst - this version is fine

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_testsrc: correct colors for smptebars

2015-08-09 Thread Kieran Kunhya
On 9 August 2015 at 12:20, Paul B Mahol one...@gmail.com wrote: --- libavfilter/vsrc_testsrc.c | 36 +++- 1 file changed, 23 insertions(+), 13 deletions(-) I'll test the SD version on the scope on Monday (cc'ing dericed who can probably test HD) Kieran.

Re: [FFmpeg-devel] FFmpeg/MPlayer/rtmpdump possibly searching for a new server and hosting

2015-07-17 Thread Kieran Kunhya
I like kieranks' offer. although i dont know the service hours or timeframe of hosting. So the machine sits in our datacentre next to my office, has a 100meg connection. Plenty more spare machines here ffmpeg could use but they're all a bit old. Kieran

Re: [FFmpeg-devel] FFmpeg/MPlayer/rtmpdump possibly searching for a new server and hosting

2015-07-14 Thread Kieran Kunhya
OVH has poor quality connectivity by the way - this could lead to performance issues in some countries. Why can't you use the server videolan offered? Kieran On 14 July 2015 at 21:32, Michael Niedermayer mich...@niedermayer.cc wrote: Hi all libnova (see CC:) has offered us a OVH (France)

Re: [FFmpeg-devel] FFmpeg/MPlayer/rtmpdump possibly searching for a new server and hosting

2015-07-17 Thread Kieran Kunhya
On 17 July 2015 at 22:00, Michael Niedermayer mich...@niedermayer.cc wrote: On Fri, Jul 17, 2015 at 04:16:53PM +0200, Michael Niedermayer wrote: On Fri, Jul 17, 2015 at 03:30:26PM +0200, Jean-Baptiste Kempf wrote: On 15 Jul, Michael Niedermayer wrote : longer awnser, videolan IIUC would

Re: [FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Kieran Kunhya
Yeah...sometimes it's best to just let the past lie there. I'd rather see it gone completely to be honest . Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread Kieran Kunhya
On 23 October 2015 at 00:19, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > > I'm tired of looking at it, being reminded of such things, and it has > served its purpose long ago. > > --- > src/legal | 11 --- > 1 file changed, 11 deletions(-) > > diff

[FFmpeg-devel] [PATCH] opusdec: Don't run vector_fmul_scalar on zero length arrays

2015-10-26 Thread Kieran Kunhya
From a1314d5c9774d555718bbc0a8612144c890bbc59 Mon Sep 17 00:00:00 2001 From: Kieran Kunhya <kier...@obe.tv> Date: Mon, 26 Oct 2015 22:26:35 + Subject: [PATCH] opusdec: Don't run vector_fmul_scalar on zero length arrays Fixes crashes on fuzzed files --- libavcodec/opusdec.c |2 +-

  1   2   3   4   5   6   7   8   >