Re: [FFmpeg-devel] [PATCH v4 2/4] mpegts: Stash original PTS for SCTE-35 sections for processing later

From: Devin Heitmueller <devin.heitmueller@ltnglobal.com>
To: Kieran Kunhya <kierank@obe.tv>
Cc: Devin Heitmueller <dheitmueller@ltnglobal.com>,
	FFmpeg development discussions and patches
	<ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v4 2/4] mpegts: Stash original PTS for SCTE-35 sections for processing later
Date: Thu, 10 Aug 2023 08:20:01 -0400
Message-ID: <CAHGibzFE4OuOBxiWaEgxy-cLHZZuVJcV1iY2NmXDz1CUPnUxpA@mail.gmail.com> (raw)
In-Reply-To: <CAK+ULv4PkLrBgjJXdd8J8YxWhqS1fnPfnrNHciuxKJauJSEFgg@mail.gmail.com>

On Thu, Aug 10, 2023 at 8:13 AM Kieran Kunhya <kierank@obe.tv> wrote:
> The (closest?) video PTS is even worse than the last PCR because the VBV means the closest PTS can be quite far from the interpolated PCR.

It's arguments like that which prompted me to explicitly exclude such
a patch from the series.  It's a discussion to be had, but not
relevant for this patch series (which makes no effort to change the
logic for how the timestamp is determined).

Wait until such a patch is submitted, and then we can debate at length
the ambiguity in the specification and what the best approach is.

Thanks,

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