Skip to content
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

ffmpeg: ignore SPS warnings when probing segments #928

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

emranemran
Copy link
Collaborator

Some input videos may not follow the h264 spec exactly - ffprobe/ffmpeg may throw warnings on such input but the videos can be decoded and played by all players. For these specific errors/warnings, we can ignore them but we also want to log other types of errors to keep track of all the different possible errors. See VID-450 for more details.

Some input videos may not follow the h264 spec exactly - ffprobe/ffmpeg
may throw warnings on such input but the videos can be decoded and
played by all players. For these specific errors/warnings, we can ignore
them but we also want to log other types of errors to keep track of all
the different possible errors. See VID-450 for more details.
@emranemran emranemran requested review from thomshutt and mjh1 October 18, 2023 06:00
@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #928 (a7e2252) into main (d7c86bf) will decrease coverage by 0.02365%.
The diff coverage is 0.00000%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##                main        #928         +/-   ##
===================================================
- Coverage   51.05068%   51.02703%   -0.02365%     
===================================================
  Files             63          63                 
  Lines           6472        6475          +3     
===================================================
  Hits            3304        3304                 
- Misses          2898        2901          +3     
  Partials         270         270                 
Files Coverage Δ
pipeline/ffmpeg.go 35.92593% <0.00000%> (-0.40366%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7c86bf...a7e2252. Read the comment docs.

Files Coverage Δ
pipeline/ffmpeg.go 35.92593% <0.00000%> (-0.40366%) ⬇️

@thomshutt thomshutt merged commit a3f5d8a into main Oct 18, 2023
12 checks passed
@thomshutt thomshutt deleted the emran/ignore-sps-warnings-during-probe branch October 18, 2023 12:01
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.

3 participants