Opened 5 years ago

Closed 5 years ago

#7524 closed enhancement (fixed)

missing PCR timestamps in secondary streams with the MPEGTS muxer

Reported by: Mark Sanders Owned by:
Priority: normal Component: avformat
Version: git-master Keywords: mpegts
Cc: neon18@nngo.net Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug:

When using the MPEG-TS muxer only the primary stream has PCR timestamps. And this behaviour can't be enhanced to mark additional substreams. It would be desirable to be able to mark other streams with PCRs.

Change History (8)

comment:1 by Mark Sanders, 5 years ago

Hi,
I came up with a solution!

The current implementation of the MPEG-TS muxer only inserts PCR timestamps in the main stream. This stream is typically the video stream in a video service, or the audio stream in a radio service.

However, when multiple streams are mixed in the same MPEG-TS service, you may need to have more than one stream with time stamps. This is true when you will remux the service to create multiple services. For example, when you copy the different audio tracks from the service for creating additional radio services. Or when more than one video stream is included and in the remux you generate independent services.

To resolve this limitation, I created on myself a patch to insert the PCR timestamps in more than the first stream. This is completely optional, as it’s not a good idea to put PCRs in all streams (even if it’s not generate errors). Two new parameters are incorporated to mark all video streams and/or all audio streams.

The patch implements this optional behavior:
https://patchwork.ffmpeg.org/patch/10827/

I hope it will accepted.
Regards.
M.Sanders.

comment:2 by Mark Sanders, 5 years ago

Hi,

Is Baptiste Coudurier <baptiste.coudurier@gmail.com> the current maintainer of the MPEG-TS muxer?

I'm still waiting for someone to check this patch.
Regards.

comment:3 by Mark Sanders, 5 years ago

Keywords: mpegtsenc added

comment:4 by Carl Eugen Hoyos, 5 years ago

Keywords: mpegtsenc pcr removed

comment:5 by Mark Sanders, 5 years ago

Hi,

A new version of patch is provided:
https://patchwork.ffmpeg.org/patch/11093/

I hope it will be accepted.
Regards.
M.Sanders.

Last edited 5 years ago by Mark Sanders (previous) (diff)

in reply to:  5 comment:6 by Mark Sanders, 5 years ago

A new version of patch is provided:
https://patchwork.ffmpeg.org/patch/11093/

Hi,

I don't received any feeback about my patch v2.

I'd like to receive some comments. Or that it was accepted.
Regards.

comment:7 by Neon, 5 years ago

Cc: neon18@nngo.net added

comment:8 by Marton Balint, 5 years ago

Resolution: fixed
Status: newclosed

You can mark streams to generate a PCR in them if you reference them in separate programs. So I think this is fixed in 88ac76be796025dfa7cc55124e881fbc59b7377a and d770e0f401e77ccd4640bbc805384f928d5de244. Reopen and update patch description if that is not enough.

Note: See TracTickets for help on using tickets.