mpegts: properly compute PTS of async KLV frames (#291) by aler9 · Pull Request #311 · bluenviron/mediacommon (original) (raw)
Copy link Copy Markdown
Member
Store last valid PTS as soon as possible, by parsing PES headers in advance, then use this PTS as timestamp of KLV frames without PTS.
Bump go-astits and fix async KLV processing logic#291
Closed
codecov Bot commented
•
edited
Loading
Copy link Copy Markdown
Codecov Report
❌ Patch coverage is 94.33962% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.11%. Comparing base (34f630d) to head (f2f2923).
⚠️ Report is 1 commits behind head on main.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| pkg/formats/mpegts/robust_demuxer.go | 95.55% | 1 Missing and 1 partial ⚠️ |
| pkg/formats/mpegts/reader.go | 50.00% | 0 Missing and 1 partial ⚠️ |
Additional details and impacted files
@@ Coverage Diff @@ ## main #311 +/- ##
Coverage 86.06% 86.11% +0.05%
Files 93 93
Lines 5990 6028 +38- Hits 5155 5191 +36
- Misses 572 573 +1
- Partials 263 264 +1
☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
`[mpegts: properly compute PTS of async KLV frames (](/bluenviron/mediacommon/pull/311/commits/f2f29233ed6c002195add4cc7da0b0dbb1320b54 "mpegts: properly compute PTS of async KLV frames (#291)
Store last valid PTS as soon as possible, by parsing PES headers in advance, then use this PTS as timestamp of KLV frames without PTS.")#291[)](/bluenviron/mediacommon/pull/311/commits/f2f29233ed6c002195add4cc7da0b0dbb1320b54 "mpegts: properly compute PTS of async KLV frames (#291)
Store last valid PTS as soon as possible, by parsing PES headers in advance, then use this PTS as timestamp of KLV frames without PTS.") `
[f2f2923](/bluenviron/mediacommon/pull/311/commits/f2f29233ed6c002195add4cc7da0b0dbb1320b54)
Store last valid PTS as soon as possible, by parsing PES headers in advance, then use this PTS as timestamp of KLV frames without PTS.
7 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})