Re: [FFmpeg-devel] avformat/mxfenc: SMPTE RDD 48:2018 Amd 1:2022 (FFV1 in MXF) support

2023-01-29 Thread Dave Rice
issue >> because we wouldn't be sure for 1 of the 2. > > I suspect we are lucky because 720i doesn't really exist in the real > world, and 576i and 480i are both multiples of 32. > > IMO mxfenc shouldn't lie, but looking at S377m StoredWidth/Height are > "best eff

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfdec: SMPTE RDD 48:2018 support

2022-07-13 Thread Dave Rice
V1SubDescriptor), > FFV1SubDescriptor }, > { { > 0x06,0x0e,0x2b,0x34,0x02,0x53,0x01,0x01,0x0d,0x01,0x01,0x01,0x01,0x01,0x3A,0x00 > }, mxf_read_track, sizeof(MXFTrack), Track }, /* Static Track */ > { { > 0x06,0x0e,0x2b,0x34,0x02,0x53,0x01,0x01,0x0d,0x01,0x01,0x01,0x01,0x0

Re: [FFmpeg-devel] [PATCH v2] avcodec: Add dv marker bsf

2022-03-14 Thread Dave Rice
> On Mar 12, 2022, at 1:09 PM, Michael Niedermayer > wrote: > > On Sat, Mar 12, 2022 at 10:11:52AM -0500, Dave Rice wrote: >> >> >>> On Mar 10, 2022, at 4:41 AM, Tobias Rapp wrote: >>> >>> On 09/03/2022 19:18, Michael Niederm

Re: [FFmpeg-devel] [PATCH v2] avcodec: Add dv marker bsf

2022-03-12 Thread Dave Rice
...] > The filter options look nice to me now. Have not actually tested the > bitstream filter on DV files, though. I tested this and this works well for me. Here's a few samples that demonstrate the filter: ./ffmpeg -i https://samples.ffmpeg.org/archive/au

[FFmpeg-devel] Mailing list conduct

2021-09-22 Thread Dave Rice
r fair critical reviews, and work collaboratively towards rough consensus. Kind Regards, The CC James Almer Thilo Borgmann Carl Eugen Hoyos Jean-Baptiste Kempf Dave Rice ___ ffmpeg-devel mailing list ffmpeg

Re: [FFmpeg-devel] [PATCH] avformat/dv: always set audio packet duration

2021-09-06 Thread Dave Rice
ted this patch and it does correct some errors. For instance, before this patch frame 3 of the output only offers half of the needed duration, so the resulting audio track is (1001/3)/2 shorter than the video. But after the patch, the video and audio outputs have the same length. ffmpeg -i

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2021-08-22 Thread Dave Rice
Hi Marton, > On Feb 23, 2021, at 3:07 PM, Dave Rice wrote: > >> On Feb 23, 2021, at 2:42 PM, Marton Balint wrote: >> >> On Sat, 20 Feb 2021, Dave Rice wrote: >> >>> Hi, >>> >>>> On Oct 31, 2020, at 5:15 PM, Marton Balint >&g

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2021-02-23 Thread Dave Rice
> On Feb 23, 2021, at 2:42 PM, Marton Balint wrote: > > > > On Sat, 20 Feb 2021, Dave Rice wrote: > >> Hi, >> >>> On Oct 31, 2020, at 5:15 PM, Marton Balint >> <mailto:c...@passwd.hu>> wrote: >>> On Sat, 31 Oct 2020, Da

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2021-02-20 Thread Dave Rice
Hi, > On Oct 31, 2020, at 5:15 PM, Marton Balint <mailto:c...@passwd.hu>> wrote: > On Sat, 31 Oct 2020, Dave Rice wrote: >>> On Oct 31, 2020, at 3:47 PM, Marton Balint >> <mailto:c...@passwd.hu>> wrote: >>> On Sat, 31 Oct 2020, Dave Rice wrote: &g

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-12-03 Thread Dave Rice
> On Nov 14, 2020, at 7:14 PM, Marton Balint wrote: > > On Fri, 6 Nov 2020, Michael Niedermayer wrote: > >> On Wed, Nov 04, 2020 at 10:44:56PM +0100, Marton Balint wrote: >>> >>> >>> On Wed, 4 Nov 2020, Michael Niedermayer wrote: >>> we have "millisecond" based formats, rounded

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-11-09 Thread Dave Rice
jittering / some droped samples ? > Its important to know as in one instance its the timestamps that need > adjustment > while in the other the samples need adjustment > ATM its down to the user to figure out on a file by file base how to deal or > ignore this. Instead it should be pos

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-11-01 Thread Dave Rice
divergence as percentage of frame duration NTSC 48000 | 0.3742511235 NTSC 44100 | 0.3869807536 NTSC 32000 | 0.3929636797 PAL 48000 | 0.3125 PAL 44100 | 0.3117913832 PAL 32000 | 0.3125 0.3929636797 is the max divergence, at least according to the spec’s limit of the allowable accumulated dif

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-10-31 Thread Dave Rice
> On Oct 31, 2020, at 5:15 PM, Marton Balint wrote: > On Sat, 31 Oct 2020, Dave Rice wrote: >>> On Oct 31, 2020, at 3:47 PM, Marton Balint wrote: >>> On Sat, 31 Oct 2020, Dave Rice wrote: >>>> Hi Marton, >>>>> On Oct 31, 2020, at 12:5

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-10-31 Thread Dave Rice
> On Oct 31, 2020, at 3:47 PM, Marton Balint wrote: > On Sat, 31 Oct 2020, Dave Rice wrote: > >> Hi Marton, >> >>> On Oct 31, 2020, at 12:56 PM, Marton Balint wrote: >>> Fixes out of sync timestamps in ticket #8762. >> >> Although Micha

Re: [FFmpeg-devel] [PATCH] avformat/dv: fix timestamps of audio packets in case of dropped corrupt audio frames

2020-10-31 Thread Dave Rice
s:0 ts: 0.88 > ret: 0 st: 0 flags:1 dts: 0.88 pts: 0.88 pos:3168000 > size:144000 > ret: 0 st: 0 flags:1 ts:-0.24 > ret: 0 st: 0 flags:1 dts: 0.00 pts: 0.00 pos: 0 > size:144000 > -ret: 0 st: 1 flags:0 ts: 2.6716

Re: [FFmpeg-devel] [PATCH 1/2] avformat/dv: allow returning damaged audio

2020-09-06 Thread Dave Rice
> On Aug 3, 2020, at 5:16 PM, Michael Niedermayer > wrote: > > On Mon, Aug 03, 2020 at 10:38:21PM +0200, Marton Balint wrote: >> >> >> On Sun, 2 Aug 2020, Dave Rice wrote: >> >>> >>> >>>> On Aug 1, 2020, at 5:26 PM, Marton

Re: [FFmpeg-devel] [PATCH 1/2] avformat/dv: allow returning damaged audio

2020-08-02 Thread Dave Rice
> On Aug 1, 2020, at 5:26 PM, Marton Balint wrote: > > > > On Sat, 1 Aug 2020, Michael Niedermayer wrote: > >> On Sat, Aug 01, 2020 at 07:28:53PM +0200, Marton Balint wrote: >>> >>> >>> On Sat, 1 Aug 2020, Michael Niedermayer wrote: >>> Fixes: Ticket8762 Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH] avformat/mov: fix memleak

2020-06-28 Thread Dave Rice
Hi Andreas, > On Jun 27, 2020, at 5:09 PM, Andreas Rheinhardt > wrote: > > Zhao Zhili: >> Remove the check on dv_demux since dv_fctx will leak if allocate >> dv_demux failed. >> --- >> libavformat/mov.c | 7 +++ >> 1 file changed, 3 insertions(+), 4 deletions(-) >> >> diff --git

Re: [FFmpeg-devel] [PATCH v1] avfilter: add colorstats, colorrgbstats, coloryuvstats video filter

2019-12-30 Thread Dave Rice
> On Dec 27, 2019, at 10:49 AM, Paul B Mahol wrote: > > That is because signalstats is doing more stuff. signalstats includes options to disable some of the calculations, possibly this could be extended to enable or disable the ones you want. It would be interesting to merge these ideas

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 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

  1   2   >