Re: [FFmpeg-devel] ? Inserting an image in the middle of a video stream

2018-04-25 Thread John P Poet
On Wed, Apr 25, 2018 at 4:25 PM Devin Heitmueller < dheitmuel...@kernellabs.com> wrote: > Hi John, > > On Wed, Apr 25, 2018 at 5:11 PM, John P Poet wrote: > > I am working on a project where I need to insert a repeating image (10 > > seconds) in the middle o

[FFmpeg-devel] ? Inserting an image in the middle of a video stream

2018-04-25 Thread John P Poet
Hi all, I am working on a project where I need to insert a repeating image (10 seconds) in the middle of a video stream while also transcoding from one video codec to another. The images are going to be either png or jpg images. I have a frame number within the video stream which tells me where

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-07-07 Thread John P Poet
On Wed, Jul 5, 2017 at 3:30 PM Marton Balint wrote: > > On Fri, 9 Jun 2017, Devin Heitmueller wrote: > > > Hello Marton, > > > > On Tue, Jun 6, 2017 at 5:45 PM, Marton Balint wrote: > > > >> As far as I remember multiple side data of the same type is not > something we > >> wanted to support. Wh

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-07-05 Thread John P Poet
On Fri, Jun 9, 2017 at 11:41 AM Devin Heitmueller < dheitmuel...@kernellabs.com> wrote: > Hello Marton, > > On Tue, Jun 6, 2017 at 5:45 PM, Marton Balint wrote: > > > As far as I remember multiple side data of the same type is not > something we > > wanted to support. Why do you need it? Can't a

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-06-07 Thread John P Poet
On Tue, Jun 6, 2017 at 4:48 PM John P Poet wrote: > On Tue, Jun 6, 2017 at 4:40 PM Kieran Kunhya wrote: > >> > >> > The cc_count is only 5 bits, which mean that only 31 3-byte "closed >> caption >> > constructs" will fit in a "block".

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-06-06 Thread John P Poet
On Tue, Jun 6, 2017 at 4:40 PM Kieran Kunhya wrote: > > > > The cc_count is only 5 bits, which mean that only 31 3-byte "closed > caption > > constructs" will fit in a "block".Testing this with 1080i60 > material, I > > found that 2 or 3 blocks was often necessary to hold all of the CC data.

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-06-06 Thread John P Poet
On Tue, Jun 6, 2017 at 3:59 PM Hendrik Leppkes wrote: > On Tue, Jun 6, 2017 at 11:45 PM, Marton Balint wrote: > > > > On Tue, 6 Jun 2017, John Poet wrote: > > > >> ff_mpeg1_encode_picture_header: Add support for AV_FRAME_DATA_A53_CC > >> frame: Add av_frame_get_side_data_at() to allow retrival o

[FFmpeg-devel] [PATCH] Add A53 closed captions to mpeg picture header.

2017-03-21 Thread John P Poet
If AV_FRAME_DATA_A53_CC "user data" is available, encode it into the mpeg picture header. This follows the spec in section 6.2.3 of http://atsc.org/wp-content/uploads/2015/03/a_53-Part-4-2009.pdf I am working on gstreamer side, to inject the information into the ffmpeg side data, and have it gene