-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle PTS rollovers #130
Comments
Some relevant threads related to timestamp rollover as I find them:
|
It looks like the answer is to detect discontinuity and add |
MPEG-TS stores packet positions using 33 bits, which means that for long running streams the position will rollover approximately every 26.5 hours. TV Kitchen appliances need to be able to ingest videos and streams of any length, which means when rollovers occur the appliance needs to properly update the origin times to account for the rollover. Issue #130
MPEG-TS stores packet positions using 33 bits, which means that for long running streams the position will rollover approximately every 26.5 hours. TV Kitchen appliances need to be able to ingest videos and streams of any length, which means when rollovers occur the appliance needs to properly update the origin times to account for the rollover. Issue #130
The new appliance-core fixes a bug where video ingestion would fail to update timestamps after 26.5 hours. Issue #130
Task
Description
MPEG PTS and DTS have a limit, and when that limit is reached they roll over.
I'm not certain if the rollover is part of the MPEGTS standard (e.g. it will always roll over at
2^33
) or if it is up to the encoding agent to determine when to initiate a rollover. Handling rollovers is non-trivial for that reason, so this issue will require a bit of R&D before implementation.The solution needs to work for stream and file based videos (e.g. a file with 2 weeks of video would have a rollover as well). This means that the rollover should be in terms of the
origin
that is passed to the video ingestion appliance (as opposed to some kind of logic likeif PTS went down, use the current time as origin
).The text was updated successfully, but these errors were encountered: