From: Devin Heitmueller <devin.heitmueller@ltnglobal.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 0/5] Properly handle CEA-708 caption data when transcoding
Date: Thu, 30 Mar 2023 09:14:50 -0400
Message-ID: <CAHGibzHz_XjM4rrhxK+D1fK6e03PFfGer0q4M290XeFuEMAzKA@mail.gmail.com> (raw)
In-Reply-To: <GV1P250MB07371EC2DB9E3E5D1590A7CA8F8E9@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM>
On Wed, Mar 29, 2023 at 8:46 PM Andreas Rheinhardt
<andreas.rheinhardt@outlook.com> wrote:
>
> Devin Heitmueller:
> > This patch series is intended to address long-standing known issues where
> > CEA-708 caption data is either duplicated or corrupted (i.e. 50% of the data
> > is lost). We employ an intermediate queue, which is available to filters that
> > change the frame rate of the video, which stashes the 608/708 data and
> > reconstructs properly formed 708 tuples on the output side (injecting
> > appropriate padding as needed).
> >
> > The common functions are implemented in libavutil, but I am open to
> > suggestions. While all the users in this patch series are within
> > libavfilter, there are use cases where this functionality would
> > be useful within libavformat (which is why libavutil was chosen).
> >
>
> If it is not used now outside of lavfi, it should be in lavfi to allow
> to change it when it turns out that changes are needed.
So I was thinking along the same lines. This would allow the API to
mature and change as needed without worrying about ABI breakage, which
would be useful since the API is relatively new and might need
tweaking.
Thanks for the suggestion. I'll move it into libavfilter in the next
patches I submit (pending any other feedback/comments).
Devin
--
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com e: devin.heitmueller@ltnglobal.com
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
prev parent reply other threads:[~2023-03-30 13:15 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-27 16:46 Devin Heitmueller
2023-03-27 16:47 ` [FFmpeg-devel] [PATCH 1/5] ccfifo: Properly handle CEA-708 captions through framerate conversion Devin Heitmueller
2023-03-27 16:47 ` [FFmpeg-devel] [PATCH 2/5] vf_fps: properly preserve CEA-708 captions Devin Heitmueller
2023-03-27 16:47 ` [FFmpeg-devel] [PATCH 3/5] yadif: Properly preserve CEA-708 closed captions Devin Heitmueller
2023-03-27 16:47 ` [FFmpeg-devel] [PATCH 4/5] tinterlace: " Devin Heitmueller
2023-03-27 16:47 ` [FFmpeg-devel] [PATCH 5/5] vf_ccrepack: Add new filter to repack CEA-708 side data Devin Heitmueller
2023-03-30 0:46 ` [FFmpeg-devel] [PATCH 0/5] Properly handle CEA-708 caption data when transcoding Andreas Rheinhardt
2023-03-30 13:14 ` Devin Heitmueller [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAHGibzHz_XjM4rrhxK+D1fK6e03PFfGer0q4M290XeFuEMAzKA@mail.gmail.com \
--to=devin.heitmueller@ltnglobal.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git