Re: [FFmpeg-devel] [PATCH] avformat/movenc: always write a colr atom

2019-07-16 Thread Dave Rice
Hi James, > On Jul 16, 2019, at 9:47 AM, James Almer wrote: > > -{ "write_colr", "Write colr atom (Experimental, may be renamed or > changed, do not use from scripts)", 0, AV_OPT_TYPE_CONST, {.i64 = > FF_MOV_FLAG_WRITE_COLR}, INT_MIN, INT_MAX, AV_OPT_FLAG_ENCODING_PARAM, > "movflags" },

Re: [FFmpeg-devel] [PATCHv3 1/2] ffplay: options to specify window position

2018-10-04 Thread Dave Rice
From caa816d70e69f85d49556ff341addab24ebcd942 Mon Sep 17 00:00:00 2001 From: Dave Rice Date: Mon, 1 Oct 2018 17:07:44 -0400 Subject: [PATCH 1/2] ffplay: options to specify window position --- doc/ffplay.texi | 4 fftools/ffplay.c | 6 +- 2 files changed, 9 insertions(+), 1 deletion

Re: [FFmpeg-devel] [PATCH 2/2] avdevice/sdl2 : add option to set window position

2018-10-03 Thread Dave Rice
> On Oct 2, 2018, at 1:32 AM, Gyan wrote: > > On Tue, Oct 2, 2018 at 2:47 AM Dave Rice wrote: > >> Allows arrangement of multiple windows such as: >> ffmpeg -re -f lavfi -i mandelbrot -f sdl -window_x 1 -window_y 1 >> mandelbrot -vf waveform,format=yuv420p -f

[FFmpeg-devel] [PATCHv2 1/2] ffplay: options to specify window position

2018-10-03 Thread Dave Rice
Thanks Marton for comments. Here is a revision to the first patch. From 3fe6a9e5279a280af9a06843621737ddc44529cc Mon Sep 17 00:00:00 2001 From: Dave Rice Date: Mon, 1 Oct 2018 17:07:44 -0400 Subject: [PATCHv2 1/2] ffplay: options to specify window position --- doc/ffplay.texi | 4

[FFmpeg-devel] [PATCH 2/2] avdevice/sdl2 : add option to set window position

2018-10-01 Thread Dave Rice
00438983c96b5db227b9975a2c160fc6aac5219d Mon Sep 17 00:00:00 2001 From: Dave Rice Date: Mon, 1 Oct 2018 17:08:35 -0400 Subject: [PATCH 2/2] avdevice/sdl2 : add option to set window position --- libavdevice/sdl2.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavdevice/sdl2.c b/libavdevice/sdl2.c

[FFmpeg-devel] [PATCH 1/2] ffplay: options to specify window position

2018-10-01 Thread Dave Rice
From 14d6833b564bd672613d50ecc4c3ede1768eee37 Mon Sep 17 00:00:00 2001 From: Dave Rice Date: Mon, 1 Oct 2018 17:07:44 -0400 Subject: [PATCH 1/2] ffplay: options to specify window position --- fftools/ffplay.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/fftools

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: add reserve free space option

2018-09-26 Thread Dave Rice
ion.h > +++ b/libavformat/version.h > @@ -33,7 +33,7 @@ > // Also please add any ticket numbers that you believe might be affected here > #define LIBAVFORMAT_VERSION_MAJOR 58 > #define LIBAVFORMAT_VERSION_MINOR 18 > -#define LIBAVFORMAT_VERSION_MICRO 100 > +#define LIBAVFORMAT_VERSION_MICRO 101 > > #define LIBAVFORMAT_VERSION

Re: [FFmpeg-devel] Register for VDD 2018 conference

2018-08-22 Thread Dave Rice
forked alternative to the work on FFV1 version 4 by the IETF cellar working group. https://twitter.com/retokromer/status/884030201050648576?s=21 Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-12 Thread Dave Rice
> On Jun 7, 2018, at 5:01 PM, Marton Balint wrote: > > On Thu, 7 Jun 2018, Dave Rice wrote: > > [...] > >> >> Before I only tested with vitc but now have a serial cable connected as well >> and found a source tape that has distinct values for LTC an

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-07 Thread Dave Rice
> On Jun 7, 2018, at 12:12 PM, Dave Rice wrote: > > >> On Jun 6, 2018, at 5:32 PM, Marton Balint wrote: >> >> On Wed, 6 Jun 2018, Dave Rice wrote: >> >>>> On Jun 6, 2018, at 4:50 PM, Marton Balint wrote: >>>> On Mon, 4 Jun 2018, Dave

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-07 Thread Dave Rice
> On Jun 6, 2018, at 5:32 PM, Marton Balint wrote: > > On Wed, 6 Jun 2018, Dave Rice wrote: > >>> On Jun 6, 2018, at 4:50 PM, Marton Balint wrote: >>> On Mon, 4 Jun 2018, Dave Rice wrote: >>>>>> In my testing the timecode value set here has

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-06 Thread Dave Rice
> On Jun 6, 2018, at 4:50 PM, Marton Balint wrote: > > On Mon, 4 Jun 2018, Dave Rice wrote: > >> >>>> In my testing the timecode value set here has corrected been associated >>>> with the first video frame (maintaining the timecode-to-first-frame

Re: [FFmpeg-devel] [PATCH 1/2] avdevice/decklink_dec: use a custom memory allocator

2018-06-05 Thread Dave Rice
> On Jun 5, 2018, at 1:17 PM, Marton Balint wrote: > > On Tue, 5 Jun 2018, Dave Rice wrote: > >>> On Jun 4, 2018, at 4:21 PM, Marton Balint wrote: >>> >>> The default memory allocator is limited in the max number of frames >>> available, &g

Re: [FFmpeg-devel] [PATCH 1/2] avdevice/decklink_dec: use a custom memory allocator

2018-06-05 Thread Dave Rice
> On Jun 4, 2018, at 4:21 PM, Marton Balint wrote: > > The default memory allocator is limited in the max number of frames available, > and therefore caused frame drops if the frames were not freed fast enough. I’ve been testing this patchset today. Yesterday I was occasionally getting

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-04 Thread Dave Rice
> On Jun 4, 2018, at 12:24 PM, Marton Balint wrote: > > On Fri, 1 Jun 2018, Dave Rice wrote: > >>> On May 31, 2018, at 5:29 PM, Marton Balint wrote: >>> On Thu, 31 May 2018, Jonathan Morley wrote: >>>> Thank you for the clarification, Dave. It m

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-06-01 Thread Dave Rice
> On May 31, 2018, at 5:29 PM, Marton Balint wrote: > > On Thu, 31 May 2018, Jonathan Morley wrote: > >> Thank you for the clarification, Dave. It might be that the Blackmagic >> approach to collecting timecode doesn’t work for that one source because it >> is in the horizontal space (HANC)

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-31 Thread Dave Rice
I will add that to the next patch, but it might be a minute before I get to > it. Thank you again for taking a look. > > I guess we got really lucky in our use case. > > Thanks, > Jon > >> On May 31, 2018, at 12:39 PM, Dave Rice wrote: >> >> Hi Jonathan,

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-31 Thread Dave Rice
nges either. I’ve replicated it with Matroska which simply moves the timecode into metadata as a string. > Please keep me posted. Meanwhile I will look into what could be causing all > the error chatter. Thanks! > Thanks, > Jon > >> On May 31, 2018, at 7:59 AM, Dave Rice

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-31 Thread Dave Rice
op Frame and all the ffmpeg captures are Drop Frame, so one has to be wrong. I’ll try to find a source that can provide a known df and ndf signal to determine which is correct. Thanks for the update, Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-25 Thread Dave Rice
> On May 25, 2018, at 17:15, Jonathan Morley wrote: > > Thank you for trying it out, Dave. > > 1) Can you please tell me more about how you tested? J30 digibeta deck playing a Betacam SP with SDI to an Ultrastudio 3D with ffmpeg with your patch and configured with

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-25 Thread Dave Rice
> On May 25, 2018, at 1:08 PM, Jonathan Morley wrote: > >> On May 25, 2018, at 8:32 AM, Marton Balint > > wrote: >> >> On Fri, 25 May 2018, Jonathan Morley wrote: >> >>> I apologize in advance for the formatting of this message.

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread Dave Rice
y finds consensus to appoint a CoC committee and as needed the CoC committee finds consensus (as a small group) on how to respond to concerns from the community and to implement the CoC. b) the larger ffmpeg community finds consensus on how to implement the CoC directly each time there’s a c

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-28 Thread Dave Rice
> On Mar 27, 2018, at 5:16 PM, wm4 <nfx...@googlemail.com> wrote: > > On Tue, 27 Mar 2018 16:45:23 -0400 > Dave Rice <d...@dericed.com <mailto:d...@dericed.com>> wrote: > >>> On Mar 27, 2018, at 4:33 PM, wm4 <nfx...@googlemail.com> wrote: >

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Dave Rice
> On Mar 27, 2018, at 4:33 PM, wm4 <nfx...@googlemail.com> wrote: > > On Tue, 27 Mar 2018 16:11:11 -0400 > Dave Rice <d...@dericed.com> wrote: > >>> On Mar 27, 2018, at 4:01 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> >>> wrote: >&g

Re: [FFmpeg-devel] [RFC] Exporting virtual timelines as stream side data

2018-03-27 Thread Dave Rice
ns to apply timelines themselves - and I figured > some would argue that point. +1 I’m willing to contribute what information or samples would be needed to help with Matroska support with virtual timelines. IMO, this would be a valuable feature to have in ffmpeg. Dave Rice __

Re: [FFmpeg-devel] [Cellar] [PATCH] avcodec/ffv1: Support for RGBA64 and GBRAP16

2018-02-05 Thread Dave Rice
ery close to a last call and submission to the IESG (chairs are welcome to offer other opinions ;) ). So while I encourage resolution to these pull requests, it seems we have a good system to manage concurrent work on both FFV1 goals of the charter. Kind Regards, Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: use sigterm_handler with sigpipe

2018-01-18 Thread Dave Rice
t thing with > SIGPIPE. This patch has been working for me and ffmpeg terminates cleanly with SIGPIPE with a valid output (moov atom written with mov or cues/seekhead written with mkv). Not sure if I understand well the disadvantage of this patch. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] ffmpeg.c: use sigterm_handler with sigpipe

2018-01-11 Thread Dave Rice
From 0faa2954010feb8428542fc33aa81e898a280c88 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 11 Jan 2018 15:52:36 -0500 Subject: [PATCH] ffmpeg.c: use sigterm_handler with sigpipe Based on a suggestion by Moritz Barsnick at http://ffmpeg.org/pipermail/ffmpeg-use

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showwaves: add draw grid support

2017-11-21 Thread Dave Rice
=3) { > +for (s = 0; s < pixstep; s++) { > +out->data[0][(yskip + j * ystep - 1) * > out->linesize[0] + x * pixstep + s] = showwaves->grid_rgba[s]; > +} > +} > + > +ysk

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

2017-11-20 Thread Dave Rice
> On Nov 20, 2017, at 11:22 AM, Derek Buitenhuis <derek.buitenh...@gmail.com> > wrote: > > On 11/20/2017 3:19 PM, Dave Rice wrote: >> TN2162 requires a colr atom for uncompressed yuv (including v210, v308, >> v408, etc) in mov, so I'd prefer to write it in thi

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

2017-11-20 Thread Dave Rice
> On Nov 20, 2017, at 9:55 AM, Derek Buitenhuis <derek.buitenh...@gmail.com> > wrote: > > On 11/20/2017 2:50 PM, Dave Rice wrote: >> I am hesitant to see write_colr option removed since the guesswork used here >> https://github.com/FFmpeg/FFmpeg/blob/8f4702a93f

Re: [FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

2017-11-20 Thread Dave Rice
> On Nov 20, 2017, at 9:01 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > > 2017-11-20 2:24 GMT+01:00 James Almer <jamr...@gmail.com > <mailto:jamr...@gmail.com>>: >> On 11/18/2017 11:19 PM, Dave Rice wrote: >>> From 41da5e48f8788b85dd7a382030bb2

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: add queue option

2017-11-19 Thread Dave Rice
vf 'tile=1x8:overlap=7:queue=1' filmstrip.mkv @end example Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 4/4] fate: add v210 mov test

2017-11-18 Thread Dave Rice
From fe9e3aa13ecf3b4cb81f279696bcc21602662e7a Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Sat, 18 Nov 2017 20:32:33 -0500 Subject: [PATCH 4/4] fate: add v210 mov test --- tests/fate/vcodec.mak | 4 tests/ref/vsynth/vsynth1-mov-v210 | 4 2 files c

[FFmpeg-devel] [PATCH 1/4] avformat/movenc: correct ImageDescription for uncompressed ycbcr

2017-11-18 Thread Dave Rice
26d9ca470f104d8448000b13c2cc97b8fc5c15ba Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 16 Nov 2017 11:53:32 -0500 Subject: [PATCH 1/4] avformat/movenc: correct ImageDescription for uncompressed ycbcr Per https://developer.apple.com/library/content/technotes/tn2162/_index.html . --- libav

[FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

2017-11-18 Thread Dave Rice
From 41da5e48f8788b85dd7a382030bb2866c506cc03 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Sat, 18 Nov 2017 20:31:27 -0500 Subject: [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 C

[FFmpeg-devel] [PATCH 2/4] avformat/movenc: write clap atom for uncompressed yuv in mov

2017-11-18 Thread Dave Rice
From e9079167c199791e652fe9cd3550333b819a8a9a Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 16 Nov 2017 11:29:06 -0500 Subject: [PATCH 2/4] avformat/movenc: write clap atom for uncompressed yuv in mov fixes 6145 --- libavformat/movenc.c | 19 +++

Re: [FFmpeg-devel] [PATCH] avformat/movenc: correct ImageDescription depth for v210 v410

2017-11-16 Thread Dave Rice
> On Nov 16, 2017, at 6:08 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > > 2017-11-16 17:54 GMT+01:00 Dave Rice <d...@dericed.com>: > >> +if (track->mode == MODE_MOV && track->par->codec_id == AV_CODEC_ID_V410) >> +avio_wb1

Re: [FFmpeg-devel] [PATCH] movenc: write clap tag

2017-11-16 Thread Dave Rice
> On Nov 16, 2017, at 11:30 AM, Dave Rice <d...@dericed.com> wrote: > >> On Jul 9, 2017, at 7:26 PM, Dave Rice <d...@dericed.com> wrote: >> >>> On Jul 7, 2017, at 7:06 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> >>> wrote: >&g

[FFmpeg-devel] [PATCH] avformat/movenc: correct ImageDescription depth for v210 v410

2017-11-16 Thread Dave Rice
162/_index.html>. From 47def189b270ac93245e002580463b254daf8484 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 16 Nov 2017 11:53:32 -0500 Subject: [PATCH] avformat/movenc: correct ImageDescription depth for v210 v410 Per https://developer.apple.com/library/conte

Re: [FFmpeg-devel] [PATCH] movenc: write clap tag

2017-11-16 Thread Dave Rice
> On Jul 9, 2017, at 7:26 PM, Dave Rice <d...@dericed.com> wrote: > > >> On Jul 7, 2017, at 7:06 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> >> wrote: >> >> On 7/7/2017 10:13 PM, James Almer wrote: >>> Isn't this necessary on

Re: [FFmpeg-devel] [PATCH RFC] libavdevice/decklink: Add support for EIA-708 output over SDI

2017-10-18 Thread Dave Rice
ter.com/ajaprez/status/910100436224499713 <https://twitter.com/ajaprez/status/910100436224499713>. I’m glad to hear that handling other VANC data is in the works, I’m particularly interested in VITC and EIA-608 with inputs. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-18 Thread Dave Rice
> On Oct 18, 2017, at 3:07 PM, Marton Balint <c...@passwd.hu> wrote: > > On Mon, 16 Oct 2017, Dave Rice wrote: > >> Hi, >> >> I tested this with my Ultrastudio Express and confirmed that I'm getting >> higher bit depth recordings with the abitscope fi

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-18 Thread Dave Rice
> On Oct 17, 2017, at 3:30 PM, Douglas Marsh <ffm...@dx9s.net> wrote: > > On 2017-10-17 09:10, Dave Rice wrote: > >>>> -audio_depth .D.. audio bitdepth (from 0 to 1) >>>> (default 16bits) >>>>16bits

Re: [FFmpeg-devel] decklink 24/32 bit question

2017-10-17 Thread Dave Rice
to cover that. The decklink sdk only defines two BMDAudioSampleType values: bmdAudioSampleType16bitInteger and bmdAudioSampleType32bitInteger. I don't think there's an easy way to support a 24 bit input here. Generally in this case I've used bmdAudioSampleType32bitInteger and then encode it at pcm_s24le. Dave Rice _

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-17 Thread Dave Rice
> On Oct 17, 2017, at 11:48 AM, Paul B Mahol <one...@gmail.com> wrote: > > On 10/17/17, Moritz Barsnick <barsn...@gmx.net> wrote: >> On Tue, Oct 17, 2017 at 11:12:46 -0400, Dave Rice wrote: >> >>> Thanks for pointing me in the right direction. I am att

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-17 Thread Dave Rice
Hi Moritz, > On Oct 17, 2017, at 10:40 AM, Moritz Barsnick <barsn...@gmx.net> wrote: > > On Tue, Oct 17, 2017 at 09:22:27 -0400, Dave Rice wrote: >>>> --- a/libavdevice/decklink_dec_c.c >>>> +++ b/libavdevice/decklink_dec_c.c >>>&

Re: [FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-17 Thread Dave Rice
Hi Paul, > On Oct 16, 2017, at 3:56 PM, Paul B Mahol <one...@gmail.com> wrote: > > On 10/16/17, Dave Rice <d...@dericed.com <mailto:d...@dericed.com>> wrote: >> Hi, >> >> I tested this with my Ultrastudio Express and confirmed that I'm getting >

[FFmpeg-devel] [PATCH] libavdevice/decklink: 32 bit audio support

2017-10-16 Thread Dave Rice
://trac.ffmpeg.org/ticket/6708 and is partly based upon Georg Lippitisch's earlier draft at https://ffmpeg.org/pipermail/ffmpeg-devel/2015-January/167649.html. From fbf2bd40471c8fa35374bb1a51c51a3f4f36b992 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 12 Oct 2017 13:40:59 -0400 S

[FFmpeg-devel] [PATCH] libavcodec: fix field_order labelling

2017-08-12 Thread Dave Rice
t using “top first (interleaved)” for TB and “bottom first (interleaved)” for BT. Comments? From de871b3fa891fa0ae6856461c1f8305cc889cde7 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Sat, 12 Aug 2017 12:30:43 -0400 Subject: [PATCH] libavcodec: fix field_order labelling

Re: [FFmpeg-devel] [PATCH] ffmpeg: fix setting field_order during muxing

2017-08-10 Thread Dave Rice
x.html#//apple_ref/doc/uid/DTS40013070-CH1-TNTAG10-THE__FIEL__IMAGEDESCRIPTION_EXTENSION__FIELD_FRAME_INFORMATION (Figure 4) From testing we have found that ffmpeg's behavior here is matching some Apple mov muxers in the assignment of a field order value, so the patch may be unneeded or the rea

Re: [FFmpeg-devel] [PATCH] movenc: write clap tag

2017-07-09 Thread Dave Rice
nclude the condition and resubmit. If the patch only impacts “uncompressed Y´CbCr” would any fate updates be needed? Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] movenc: write clap tag

2017-07-09 Thread Dave Rice
> On Jul 7, 2017, at 11:32 AM, Derek Buitenhuis <derek.buitenh...@gmail.com> > wrote: > > On 7/7/2017 3:20 AM, Dave Rice wrote: >> Resolves https://trac.ffmpeg.org/ticket/6145 and writes a clap atom that is >> coincident with the frames wid

[FFmpeg-devel] [PATCH] movenc: write clap tag

2017-07-06 Thread Dave Rice
Resolves https://trac.ffmpeg.org/ticket/6145 and writes a clap atom that is coincident with the frames width and height. From 23d80d0d47829fed61e817b1e7c3f6d420c9ab5c Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 6 Jul 2017 21:12:38 -0400 Subject: [PATCH] movenc:

Re: [FFmpeg-devel] [RFC][PATCH] movenc: write clap tag

2017-07-06 Thread Dave Rice
> On Jul 6, 2017, at 9:39 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> > wrote: > > On 7/7/2017 2:20 AM, Dave Rice wrote: >> Currently this patch writes the clap atom under this condition >> "track->par->codec_id == AV_CODEC_ID_RAWVIDE

[FFmpeg-devel] [RFC][PATCH] movenc: write clap tag

2017-07-06 Thread Dave Rice
her comments welcome. Thanks. From a5e399c6eeaa64aef52dc498cc62114428f42941 Mon Sep 17 00:00:00 2001 From: Dave Rice <d...@dericed.com> Date: Thu, 6 Jul 2017 21:12:38 -0400 Subject: [PATCH] movenc: write clap tag --- libavformat/movenc.c | 19 +++ 1 file changed, 19 insertion

Re: [FFmpeg-devel] [WIP][PATCH] avfilter: add video oscilloscope filter

2017-04-26 Thread Dave Rice
> On Apr 26, 2017, at 2:37 AM, Paul B Mahol <one...@gmail.com> wrote: > > On 4/26/17, Dave Rice <d...@dericed.com> wrote: >> >> >> I'm curious why floats are used for x,y coordinates within a frame rather >> than integers. > > So it the

Re: [FFmpeg-devel] [PATCH] avfilter: add pixscope filter

2017-04-25 Thread Dave Rice
> On Apr 24, 2017, at 5:03 PM, Paul B Mahol <one...@gmail.com> wrote: > > On 4/24/17, Dave Rice <d...@dericed.com> wrote: >> >>> On Apr 24, 2017, at 4:47 PM, Paul B Mahol <one...@gmail.com> wrote: >>> >>> On 4/24/17, Dave Rice <d.

Re: [FFmpeg-devel] [WIP][PATCH] avfilter: add video oscilloscope filter

2017-04-25 Thread Dave Rice
5 - s->values[i-1].p[c]; > +int y = 255 - s->values[i].p[c]; > + > +draw_line(>draw, s->ox + x, s->oy + y, s->ox + px, s->oy > + py, frame, s->colors[c]); > +} > +} > +} > + > +return

Re: [FFmpeg-devel] [PATCH] avfilter: add pixscope filter

2017-04-24 Thread Dave Rice
> On Apr 24, 2017, at 4:47 PM, Paul B Mahol <one...@gmail.com> wrote: > > On 4/24/17, Dave Rice <d...@dericed.com> wrote: >> >> I tested this filter and find it very useful, but could it be adjusted to >> handle full range video. Currently when I test wi

Re: [FFmpeg-devel] [PATCH] avfilter: add pixscope filter

2017-04-24 Thread Dave Rice
> rgba[0] : yuva[0], average[s->rgba_map[0]], min[s->rgba_map[0]], > max[s->rgba_map[0]]); > +draw_text(>draw, in, s->is_rgb ? >red : >white, X + 18, Y + > 30 + 200 + 15, text, 0); > +snprintf(text, size

Re: [FFmpeg-devel] [PATCHv3] add signature filter for MPEG7 video signature

2017-03-20 Thread Dave Rice
> On Mar 20, 2017, at 10:13 AM, Paul B Mahol <one...@gmail.com> wrote: > > On 3/20/17, Dave Rice <d...@dericed.com> wrote: >> On Jan 6, 2017, at 1:34 PM, Gerion Entrup <gerion.entrup.ff...@flump.de> >> wrote: >>> >>> On Donnerst

Re: [FFmpeg-devel] [PATCHv3] add signature filter for MPEG7 video signature

2017-03-20 Thread Dave Rice
+av_get_frame_filename(filename, sizeof(filename), sic->filename, >>> input); >> >> its more robust to use a av_assert*() on the return code if its assumed >> to be never failing than just a comment as a comment cannot be >> automatcially checked for validity currently. > I chose av_assert0 because the check is done only nb_inputs times. > > The attached patch also fixes the file writing for every frame the one input > is > longer than the other. Just bumping this thread. I've been using the patch and find it very helpful and would like to see it in libavfilter. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

2017-03-02 Thread Dave Rice
t;> Regards, >> Tobias > > Well, that makes things a bit more complex. > Maybe the Matroska spec should follow and also avoid mentioning v2? > That way we could just pretend those files don't exist and not bother > trying to drop their extradata during muxing. I agree. In the FFV1 spec, it is careful to say that it intentionally does not describe FFV1 version 2. I will submit a patch to the Matroska specification to avoid using relative language based on version 2. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams

2016-10-20 Thread Dave Rice
ems or long lines. Have rebased the patch onto > current git HEAD and attached it as a file. I tested this and the new version applied for me. The output of xml=q=1 validated against the revision. LGTM. Having this data in ffprobe is really helpful. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-20 Thread Dave Rice
quot;PixelHeight - PixelCropTop - PixelCropBottom" which is DisplayWidth=40 DisplayHeight=200 which is wrong, since the original file intended to show the cropped 40x200 image in a 320x240 display. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknown

2016-10-15 Thread Dave Rice
> On Oct 15, 2016, at 7:54 PM, Kieran O Leary wrote: > > Hi, > > On Sat, Oct 15, 2016 at 11:09 PM, James Almer wrote: > >> >> -#sar 0: 1/1 >> +#sar 0: 0/1 >> > > This commit looks very useful. A few archivists (including myself) > encountered

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms.

2016-10-13 Thread Dave Rice
> On Oct 13, 2016, at 1:56 PM, Hendrik Leppkes wrote: > > Am 13.10.2016 19:03 schrieb "liangsi" : >> >> --- >> libavformat/isom.h | 1 + >> libavformat/mov.c | 5 - >> 2 files changed, 5 insertions(+), 1 deletion(-) >> > > Can you elaborate on the

Re: [FFmpeg-devel] [PATCH] avformat/matroska: fix MatroskaVideoFieldOrder enum values

2016-10-12 Thread Dave Rice
T = 14, > } MatroskaVideoFieldOrder; > > typedef enum { > -- > 2.9.1 LGTM. Thanks for fixing this. It now aligns to the same list in QuickTime as was intended in the spec. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: fix Tags master on seekable output if there are tags after the last stream duration

2016-10-07 Thread Dave Rice
_bc, MATROSKA_ID_TAGSTRING, >> duration_string, 20); >> } >> } >> +avio_seek(mkv->tags_bc, curr, SEEK_SET); >> } >> if (mkv->tags.pos && !mkv->is_live) { >> avio_seek(pb, mkv->tags.pos, SEEK_SET); > > > Ping. It's a recent regression on a muxer, so I'd like to solve it asap. > I'll push it later today if nobody comments. I tested this and it does fix the issue so that the added "key" tag is properly formed, whereas the tag is only null bytes without the patch. Thanks! Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Dave Rice
> On Oct 6, 2016, at 4:33 PM, James Almer wrote: > > On 10/6/2016 3:28 PM, Michael Niedermayer wrote: >> On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote: >>> On 10/6/2016 7:29 AM, Michael Niedermayer wrote: On Mon, Oct 03, 2016 at 08:36:56PM -0300, James

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-05 Thread Dave Rice
ches and it works well and resolves ticket #4347. The CRC-32 Elements are in the right positions according to the latest CELLAR drafts on Matroska and the resulting files pass mkvalidator. Thanks much for this. Dave Rice ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroska: write FlagInterlaced element in WebM

2016-10-03 Thread Dave Rice
ell. I'd prefer to see a patch to webM to consider adding FieldOrder to the format rather than see FieldOrder removed from the webM muxer in FFmpeg. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Allow to disable writing the timecode track

2016-09-25 Thread Dave Rice
ing the timecode track. >> >> Fixes ticket #5492. >> --- >> doc/muxers.texi |3 +++ >> libavformat/movenc.c |6 -- >> libavformat/movenc.h |1 + >> 3 files changed, 8 insertions(+), 2 deletions(-) > > should be ok, but

Re: [FFmpeg-devel] calculation of probe score

2016-09-13 Thread Dave Rice
rmat only? > > As Michael said, yes. > Note that some containers (like H.264 Annex B or ADTS) correspond > to a codec, so for containers that "allow" random codecs (like mpeg > streams) the same probe functions that allow detecting formats can > be used to detect codecs.

Re: [FFmpeg-devel] [VDD2016] Technical Discussion Results

2016-09-12 Thread Dave Rice
yer pixel format support in FFV1 - Support for additional color information in FFV1 to better handle lossless DPX->FFV1->DPX transfer while preserving characteristics such as if the sampling is logarithmic or linear. Probably others. Feel welcome to suggest a bounty or send questions to m

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_headers_first option

2016-09-04 Thread Dave Rice
reordering would create invalid ffprobe xml. ffprobe.xsd defines the elements in a `sequence`, so if the order of elements is really arbitrary it should be changed to a `choice`. Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc: omit segment UID for webm

2016-06-22 Thread Dave Rice
> the beginning of the next element that is not a valid sub-element of > that Master-element" (from the current EBML spec) Best Regards, Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc: omit segment UID for webm

2016-06-21 Thread Dave Rice
ML Element." I think "EBML Element" in this case should be considered to mean "EBML Elements as defined by the EBML DocType". The webm EBML Schema does not define a SegmentUID Element as Matroska does, so the Element is meaningless in webm. It may happen to follow the E

[FFmpeg-devel] [PATCH 2/2] avfilter/vf_histogram: indent histogram options

2016-06-10 Thread Dave Rice
Some indentation after the previous patch. Dave Rice 0002-avfilter-vf_histogram-indent-histogram-options.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 1/2] avfilter/vf_histogram: shortcuts for histogram options

2016-06-10 Thread Dave Rice
This patch adds some shortcuts for the histogram filter's options in a way that is consistent with the waveform and vectorscope filter. Dave Rice 0001-avfilter-vf_histogram-shortcuts-for-histogram-option.patch Description: Binary data ___ ffmpeg

Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc.c: use metadata key/value to set mastering metadata

2016-06-09 Thread Dave Rice
a specification [1]. It would be better to use these defined elements from the specification rather than somehow fit this data into tags. Dave Rice [1] https://matroska.org/technical/specs/index.html#MasteringMetadata > On Tue, Apr 19, 2016 at 8:12 AM, Neil Birkbeck <neil.birkb...@gmail.c

Re: [FFmpeg-devel] [PATCH] ffprobe: Support adding av_log output to frames

2016-06-08 Thread Dave Rice
> On Jun 8, 2016, at 12:30 PM, Michael Niedermayer <mich...@niedermayer.cc> > wrote: > > On Tue, Jun 07, 2016 at 08:06:35PM -0400, Dave Rice wrote: >> >>> On May 31, 2016, at 3:23 PM, Michael Niedermayer <mich...@niedermayer.cc> >>> wrote: >&

Re: [FFmpeg-devel] [PATCH] ffprobe: Support adding av_log output to frames

2016-06-07 Thread Dave Rice
80 AC, 80 MV errors in I frame When the mpeg2video decoding log appears in sdterr, then the node is written but empty. During a frame when there is no logged message, there is no node. Also I’m uncertain what the value to use with -show_log means. I’m using -show_log 1 for now. Is

[FFmpeg-devel] [PATCH] ffprobe.xsd: add missing timecode attribute to frameSideDataType

2016-06-07 Thread Dave Rice
A timecode attribute was added to the ffprobe output of side_data. This patch updates the XSD to include the attribute as well. Dave Rice 0001-ffprobe.xsd-add-missing-timecode-attribute-to-frameS.patch Description: Binary data ___ ffmpeg-devel

[FFmpeg-devel] Matroska / FFV1 Symposium in Berlin

2016-06-03 Thread Dave Rice
...@mediaarea.net <mailto:i...@mediaarea.net>. More information and a link to the registration form are available at https://mediaarea.net/MediaConch/notimetowait.html <https://mediaarea.net/MediaConch/notimetowait.html>. Best Regards, Dave Rice [1] https://datatracker.ietf.org/wg/ce

Re: [FFmpeg-devel] [PATCH] ffprobe: Support adding av_log output to frames

2016-05-31 Thread Dave Rice
ricks or simply by assuming av_log would never be called from > another thread. > > doc/ffprobe.xsd update missing (TODO & help welcome) I'll volunteer to do the ffprobe.xsd output. Thanks much. Dave Rice > Fixes Ticket5521 > > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc.c: use metadata key/value to set mastering metadata

2016-04-03 Thread Dave Rice
001-lavf-matroskaenc.c-use-metadata-key-value-to-set-mas.patch>___ Wouldn’t it reduce confusion if the proposed MasteringMetadata elements[1] were used rather than rather than storing the same data as Matroska tags? Dave Rice [1] as discussed on the CE

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Demux the PixelCrop* values

2016-03-29 Thread Dave Rice
most happened with the "old" libavformat > rotation metadata, which is also exported as normal metadata.) > > While just adding a "hack" to export metadata for essentially 1 API > user might be acceptable if adding "proper" API is too hard for now, at &g

Re: [FFmpeg-devel] movenc: Enable timecode tracks in MP4

2016-03-15 Thread Dave Rice
e source file has timecode and the frame rate is changed that FFmpeg provide a warning, but that the encoding still occurs. Dave Rice signature.asc Description: Message signed with OpenPGP using GPGMail ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [Cellar] [Matroska-devel] Colour Format proposal

2016-02-19 Thread Dave Rice
tion/blob/master/specification.markdown#ebml-schema-element-attributes>. > Begin forwarded message: > > From: Frank Galligan <frankgalli...@gmail.com> > Subject: Re: [Cellar] [Matroska-devel] Colour Format proposal > Date: February 18, 2016 at 2:50:27 PM EST > To: Dave

[FFmpeg-devel] [PATCH] doc/filters: add example of pipe protocol use in movie filter

2016-02-12 Thread Dave Rice
Hi, Based on Paul’s advice on https://trac.ffmpeg.org/ticket/5229#comment:1. This adds an example of the pipe protocol in the movie filter with necessary escaping. Dave Rice 0001-doc-filters-add-example-of-pipe-protocol-use-in-movi.patch Description: Binary data

Re: [FFmpeg-devel] [PATCH] decklink: support all valid numbers of audio channels

2016-01-05 Thread Dave Rice
ges to decklink output code > - more stuff I don't remember right now [...] Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] clarification on General Description for FFV1 Draft Specification

2015-12-31 Thread Dave Rice
this reading it seems like 2 planes is possible; however, are 2 plane encodings possible (grayscale with alpha or Y with only Cb)? Re: "since it is most likely that the RCT will immediately be converted to RGB during decoding”. Is there anyone other conversion possible? Best Regards, Dave Rice

Re: [FFmpeg-devel] [PATCH] decklink: support all valid numbers of audio channels

2015-12-22 Thread Dave Rice
53421bc221419f2bdd1/vrecord#L275-L303. However in Media Express (the free utility that comes with Decklink products) it only places all channels in a single track with unspecified arrangement. I think mimicking Media Express is appropriate and avoid presuming too much about the audio. If neede

Re: [FFmpeg-devel] [PATCH] avfilter: add aphasemeter filter

2015-08-11 Thread Dave Rice
On Aug 11, 2015, at 2:31 PM, Paul B Mahol one...@gmail.com wrote: On 8/11/15, Paul B Mahol one...@gmail.com wrote: Hi, This should also address ticket #2824. I tested this with in-phase and out of phase audio and it worked great. I'm glad there's a way to detect this now. Thanks, Dave

Re: [FFmpeg-devel] [libav-devel] FFV1/MKV IETF Session followup

2015-07-24 Thread Dave Rice
Hi all, Further reporting on the IETF happenings below. On Jul 22, 2015, at 7:44 PM, Dave Rice d...@dericed.com wrote: Hi all, An HTML5 presentation of today's IETF meeting on FFV1 and Matroska is available at http://recordings.conf.meetecho.com/Playout/watch.jsp?recording

Re: [FFmpeg-devel] Add support for j2ki/mxf

2015-07-23 Thread Dave Rice
/. This includes 2 samples from OpenCube and 2 from SAMMA. The OpenCube files prefixed with Op1a are provided by Packed.be http://packed.be/. [..] Dave Rice ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg

[FFmpeg-devel] FFV1/MKV IETF Session followup

2015-07-22 Thread Dave Rice
that the community would need to participate in the IETF or the effort would fail. Tessa said that she hoped the community and IETF would come together. Steve Lhome, an original author of Matroska, said he would continue to participate where ever the work was going to happen. Cullen and Dave Rice pointed

Re: [FFmpeg-devel] [PATCH] conversion of FFV1 specification from lyx to markdown

2015-07-21 Thread Dave Rice
On Jul 21, 2015, at 10:24 AM, Michael Niedermayer mich...@niedermayer.cc wrote: On Mon, Jul 20, 2015 at 11:55:41PM -0400, Dave Rice wrote: On Jul 20, 2015, at 8:52 PM, Michael Niedermayer mich...@niedermayer.cc wrote: On Tue, Jul 21, 2015 at 02:14:11AM +0200, Michael Niedermayer

  1   2   >