-
Notifications
You must be signed in to change notification settings - Fork 35
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
[v4] Consider adding prediction to inter mode #170
Comments
This is duplicate, there even was GSoC to do just this. |
Where can I find an open issue for this? A GSoC wiki page for a project that never came to fruition isn't the same thing. |
@pjotrek-b's http://download.das-werkstatt.com/pb/mthk/ffv1_stats/latest/ seems to give some intuition on how the current "pseudo-inter" prediction is working:
I guess that just shows the current thing isn't enough for bigger frames. Which we kind of know already, oops. (My understanding of the two tables is that GOP size set compares GOP=1 vs 300, raw size set compares raw vs GOP=1. I hope I am reading them right.) (Also there's no raw vs GOP=300 comparison and I am too lazy to do calculations. So I also don't know whether GOP usefully compensates for the compression loss from more slices.) |
@Artoria2e5: Indeed, there are 2 different comparison tables listed:
Each with different encoding parameters. As far as I remember, the 2-pass encoding for abolishes the size benefit in favor of GOP=1, in my tests. However, the samples used in those tables are very short - and I remember the last time I've tested 2-pass with FFV1, it definitely had issues with regular/longer samples. The 2-pass logfile also became very big very fast. It would actually also be interesting, if these numbers are still valid. |
So on the GSoC front, I dug these links up:
|
There's already a dependency on a large part of the previous frame's decoding process for inter frames, and some very simple inter prediction would allow for good compression efficiency gains without a huge hit in performance.
I didn't see this discussed anywhere else, so apologies if it's a duplicate.
The text was updated successfully, but these errors were encountered: