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

Detect resolution change and reinit hw session #2884

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

leszko
Copy link
Contributor

@leszko leszko commented Oct 9, 2023

It reinitializes the hw session if there is a mismatch in media info between segments.

Note that it does not fix the issue completely, because if the resolution is changed inside a segment, then the given segment is not transcoded correctly. But it makes the issue way less problematic, becuase all consecutive segments are ok.

fix https://linear.app/livepeer/issue/VID-438/analyze-pink-and-green-lines-in-transcoded-video

@leszko leszko requested a review from thomshutt October 9, 2023 08:47
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #2884 (81b6ffc) into master (45a03e7) will increase coverage by 0.02924%.
The diff coverage is 100.00000%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #2884         +/-   ##
===================================================
+ Coverage   56.38155%   56.41079%   +0.02924%     
===================================================
  Files             89          89                 
  Lines          19384       19397         +13     
===================================================
+ Hits           10929       10942         +13     
  Misses          7849        7849                 
  Partials         606         606                 
Files Coverage Δ
server/broadcast.go 77.66831% <100.00000%> (+0.05514%) ⬆️
server/mediaserver.go 67.47536% <100.00000%> (+0.24847%) ⬆️

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 45a03e7...81b6ffc. Read the comment docs.

Files Coverage Δ
server/broadcast.go 77.66831% <100.00000%> (+0.05514%) ⬆️
server/mediaserver.go 67.47536% <100.00000%> (+0.24847%) ⬆️

Copy link
Contributor

@thomshutt thomshutt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, is there any easy way we can track how often the session reinits are happening?

@leszko
Copy link
Contributor Author

leszko commented Oct 9, 2023

LGTM, is there any easy way we can track how often the session reinits are happening?

Good point, added logging when HW Session Reinit is requested. PTAL: 81b6ffc

@thomshutt
Copy link
Contributor

LGTM, is there any easy way we can track how often the session reinits are happening?

Good point, added logging when HW Session Reinit is requested. PTAL: 81b6ffc

Looks good! 🚢 🇮🇹

@leszko leszko merged commit 75259e3 into master Oct 10, 2023
18 checks passed
@leszko leszko deleted the rafal/fix-resolution-change branch October 10, 2023 06:54
eliteprox pushed a commit to eliteprox/go-livepeer that referenced this pull request Feb 21, 2024
* Detect resolution change and reinit hw session

* Add log for requesting HW Session Reinit
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.

None yet

2 participants