[FFmpeg-devel] Re: [PATCH] avformat/mpegtsenc: Limit TS invalid messages

[Michael Niedermayer via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1136 bytes --]

Hi

On Thu, Oct 02, 2025 at 11:36:33PM +0200, David Dominguez via ffmpeg-devel wrote:
> mpegts_write_pes() warns when DTS - PCR is negative. This is
> done to detect frames that are late for decoding, meaning that
> timestamping is wrong or the muxrate value is insufficient.
> 
> But this comparison only makes sense on the first packet of
> the frame. After that the comparison will always be negative
> as DTS doesn't change until the next frame, resulting in
> hundreds of useless warnings.
> 
> Addtionaly, for long key frames that take several frame periods
> to transmity false warnings may be issued if max_delay
> is low enough. For example, in low latency CC video.
> 
> Limiting the warning to the start of frame transmission fixes
> these issues.

This doesnt feel right, the time a frame is decoded has
to be after all of its parts are transmitted
But there are other people here, who know mpeg-ts much better than i
do

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 163 bytes --]

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org