-
Notifications
You must be signed in to change notification settings - Fork 459
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
Fix freeze after ad detected, force pause/resume player #199
base: master
Are you sure you want to change the base?
Conversation
This could result in quite a few pause/play on Source quality I think? Since that code will get hit each time it picks a different quality. Though several pause/plays would be better than freezing. |
That could indeed be a problem for unstable connections. I haven't noticed anything wrong on a stable connection for what it is worth. |
The issue still happen with that change (freezing), tested on Auto and Source. I have a stable connection (desktop ethernet fiber 1gbs). Forced to pause/play very often (~40s). |
While this change helped a good bit with the video freeze during ads, I was still having the occasional freeze. I have been testing this tweak with a bit of delay for the past couple weeks and it seems to be working flawlessly for me.
Just to test if anything changed in the latest version from 3 days ago, I removed this tweak, and it went back to freezing on mid-roll ad blocking. |
I have been using this patch without a delay and it has been working fine for me for months. The issue some people have is hard to debug for me since it always works but good to see a delay works for you |
I opened an issue not realizing there is an open request here... |
Potential fix for #187 for vaft
This forces a pause/play after an ad is detected, this fixed the video freezing issue for me.