Skip to content

h264: support SEI recovery point in DTS extractor - #341

Open
keks2293 wants to merge 1 commit into
bluenviron:mainfrom
keks2293:h264-sei-recovery-point
Open

h264: support SEI recovery point in DTS extractor#341
keks2293 wants to merge 1 commit into
bluenviron:mainfrom
keks2293:h264-sei-recovery-point

Conversation

@keks2293

@keks2293 keks2293 commented Jul 9, 2026

Copy link
Copy Markdown

Treat a SEI recovery point message (payload type 6) as a random access point, similarly to a CRA in H265, so the DTS extractor can (re)initialize its state from the first coded frame that follows the recovery point.

Treat a SEI recovery point message (payload type 6) as a random
access point, similarly to a CRA in H265, so the DTS extractor can
(re)initialize its state from the first coded frame that follows the
recovery point.
@codecov

codecov Bot commented Jul 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 67.14286% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.79%. Comparing base (8f30c83) to head (cb397a8).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
pkg/codecs/h264/dts_extractor.go 65.00% 12 Missing and 2 partials ⚠️
pkg/codecs/h264/sei.go 76.00% 4 Missing and 2 partials ⚠️
pkg/codecs/h264/is_random_access.go 40.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #341      +/-   ##
==========================================
- Coverage   86.06%   85.79%   -0.27%     
==========================================
  Files          98       99       +1     
  Lines        6129     6175      +46     
==========================================
+ Hits         5275     5298      +23     
- Misses        584      603      +19     
- Partials      270      274       +4     

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant