[FFmpeg-devel] [PATCH] set bitrate from rc_max_rate for duration estimation will fix ticket #3678

2014-08-20 Thread Marc-Antoine Arnaud
From: Marc-Antoine Arnaud arnaud.marcanto...@gmail.com --- libavformat/utils.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index b4ca342..738d1f0 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2280,6 +2280,13 @@ static

Re: [FFmpeg-devel] [PATCH] set bitrate from rc_max_rate for duration estimation will fix ticket #3678

2014-08-20 Thread Marc-Antoine ARNAUD
Hi Carl, Yes I test with the attached file. The case is when the input video stream is in Mpeg2, the bitrate of the stream was setup in rc_max_rate, not in bitrate [https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/mpeg12dec.c#L1284]. For the remark you mean I can add an else if

Re: [FFmpeg-devel] [PATCH] set bitrate from rc_max_rate for duration estimation will fix ticket #3678

2014-08-20 Thread Marc-Antoine ARNAUD
Hi Michael, I agreed how we can take the information in the wrapper if it's present. In MXF case it can be done. But in other formats, this information is not necessary present ... The other possibility is to set the duration to -1 to disable print when we can't known exactly the duration.

Re: [FFmpeg-devel] [PATCH] set bitrate from rc_max_rate for duration estimation will fix ticket #3678

2014-08-21 Thread Marc-Antoine ARNAUD
I have analysed a little more of code, and in fact the duration was not present in the MXF. It's for this reason who we pass in the estimation based on the bitrate. You can see here the selection of the method: https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/utils.c#L2458 About the max

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Marc-Antoine ARNAUD
New patch version which fixe the last remark. Le ven. 10 nov. 2017 à 00:47, Michael Niedermayer <mich...@niedermayer.cc> a écrit : > On Thu, Nov 09, 2017 at 10:22:23AM +0000, Marc-Antoine ARNAUD wrote: > > Please find the merged patch in attachement. > > > > Thanks &g

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-22 Thread Marc-Antoine ARNAUD
t; On 11/22/2017 10:24 AM, Marc-Antoine ARNAUD wrote: > > New patch version which fixe the last remark. > > [...] > > > +char depth_string[8]; > > +snprintf(depth_string, sizeof(depth_string), "%d", > (1<depth) - 1); > > +

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-29 Thread Marc-Antoine ARNAUD
Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer <mich...@niedermayer.cc> a écrit : > On Wed, Nov 22, 2017 at 10:24:30AM +0000, Marc-Antoine ARNAUD wrote: > > New patch version which fixe the last remark. > > > > > > Le ven. 10 nov. 2017 à 00:47, Michael Ni

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-30 Thread Marc-Antoine ARNAUD
Le jeu. 30 nov. 2017 à 01:51, Michael Niedermayer <mich...@niedermayer.cc> a écrit : > On Wed, Nov 29, 2017 at 11:28:40AM +0000, Marc-Antoine ARNAUD wrote: > > Le mer. 22 nov. 2017 à 17:54, Michael Niedermayer <mich...@niedermayer.cc > > > > a écrit : > > >

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-09 Thread Marc-Antoine ARNAUD
Please find the merged patch in attachement. Thanks Le mer. 8 nov. 2017 à 17:12, Paul B Mahol <one...@gmail.com> a écrit : > On 11/8/17, Marc-Antoine ARNAUD <arnaud.marcanto...@gmail.com> wrote: > > This patch will fix the stride issue. > > Is it valid for you

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-08 Thread Marc-Antoine ARNAUD
3PM +0000, Marc-Antoine ARNAUD wrote: > > > > > vf_geq.c | 126 > --- > > 1 file changed, 89 insertions(+), 37 deletions(-) > > b0379f3d7f1d9660e209fe491f48fd7f70113615 > 0001-avfilt

[FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2017-11-02 Thread Marc-Antoine ARNAUD
0001-avfilter-slice-processing-for-geq.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter: slice processing for geq

2018-01-03 Thread Marc-Antoine ARNAUD
Sorry, I have upgrade and re-generate patches. Le mer. 3 janv. 2018 à 23:47, Moritz Barsnick <barsn...@gmx.net> a écrit : > On Wed, Jan 03, 2018 at 12:51:06 +0000, Marc-Antoine ARNAUD wrote: > > Changelog | 10 + &

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-15 Thread Marc-Antoine ARNAUD
ay require to patch that at an upper level than encoder to be sure all works similary. But I don't know where to be honest. Marc-Antoine Le sam. 13 oct. 2018 à 14:03, Martin Vignali a écrit : > Le mer. 10 oct. 2018 à 17:13, Marc-Antoine ARNAUD < > arnaud.marcanto...@gmail.com> a écrit

[FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-04 Thread Marc-Antoine ARNAUD
0001-prores_ks-use-CodecContext-for-color-information-if-.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-05 Thread Marc-Antoine ARNAUD
work like that for Mpeg2video codec, so we expect to have the same here. Marc-Antoine Le jeu. 4 oct. 2018 à 18:36, Paul B Mahol a écrit : > On 10/4/18, Marc-Antoine ARNAUD wrote: > > > > > > Why? > > IIRC this patch is not needed. >

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-10 Thread Marc-Antoine ARNAUD
made sense to keep that patch. Maybe with removing the `else` to don't take colorspace from pictures as other codecs can do. Marc-Antoine Le ven. 5 oct. 2018 à 10:15, Paul B Mahol a écrit : > On 10/5/18, Marc-Antoine ARNAUD wrote: > > In our case we have some files with bad

Re: [FFmpeg-devel] avcodec/prores aw improvment

2018-10-10 Thread Marc-Antoine ARNAUD
Great patches ;-) I have submit a patch too, regarding colospace in prores: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235034.html You have also in your patch 2, the same purpose (take colorspace from picture), do you have any objection to take it from the CodecContext as it works like

Re: [FFmpeg-devel] prores_ks: use CodecContext for color information if specified

2018-10-10 Thread Marc-Antoine ARNAUD
I have updated the patch with our discussion. It took information only from the codec context. Marc-Antoine Le mer. 10 oct. 2018 à 11:36, Paul B Mahol a écrit : > On 10/10/18, Marc-Antoine ARNAUD wrote: > > For me it's the only codec who use picture colorspace as source. > >

Re: [FFmpeg-devel] [PATCH] avfilter/avf_aphasemeter: Add out-of-phase and mono detection

2019-05-09 Thread Marc-Antoine ARNAUD
Few remarks: - For performances you have to align your data in the structure AudioPhaseMeterContext: don't mix float, float array and bool. Put bool at the end to align memory access. - Use complete name and it's not a setter: set_meta, name it add_metadata. - use inline functions for these