-
-
Notifications
You must be signed in to change notification settings - Fork 556
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
Olive plays videos slower than expected #23
Comments
This appears to have lessened since disabling vsync in 40213c0, however it is difficult to actually tell. As a result, I'll leave this issue open and as sound is implemented, it should become more apparent. |
On a high-end PC, I've observed videos actually playing slightly faster than usual - confirmed by comparing with VLC playback of the same video. |
With vsync re-enabled, the high-end PC still appears to play clips slightly fast making it unlikely vsync is the culprit. The culprit may instead be power saving functions employed by the laptop that this issue was first observed on. There's no reason Olive should be playing videos faster, and the strangest part is if these videos are playing slower, why is Olive not reporting them? Clearly more testing is required. |
Disabling vsync has definitely fixed the slowness on the original PC. Olive appears to consistently play videos slightly too fast now. |
This appears to no longer be a thing and does not seem to have affected audio playback |
Without reporting so (no warning about a frame taking longer than expected, i.e. as far as the app is concerned it's realtime), Olive appears to play all video slightly slower than it should.
This is likely either a vsync issue (similar to #20) or a result of Olive's single-threaded nature. Either way this will become increasingly problematic as #4 is finally implemented.
The text was updated successfully, but these errors were encountered: