-
Notifications
You must be signed in to change notification settings - Fork 253
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
Use region dimensions in SAD and ME #3097
base: master
Are you sure you want to change the base?
Conversation
fc03f98
to
e14fa64
Compare
Codecov ReportBase: 86.77% // Head: 86.80% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #3097 +/- ##
==========================================
+ Coverage 86.77% 86.80% +0.03%
==========================================
Files 83 83
Lines 33105 33098 -7
==========================================
+ Hits 28727 28732 +5
+ Misses 4378 4366 -12
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
e14fa64
to
c2c7a7a
Compare
It seems that some of the assumptions do not hold in certain circumstances. To investigate, I would revert the parts where width and height parameters are removed and assert that they match the region dimensions. |
This is going to be rather painful to debug given that it only seems to be failing on x64 Macos. Why that is, I have no clue... I'll fix it up when I have a moment. |
This avoids having to re-check bounds every time we perform SAD, as the region knows its own size. It also may save 2 usize's being passed around during ME. To enforce this, we also remove the w and h parameters from everywhere. This is part of a series of commits authored by @maj160 to improve performance of rav1e.
98994ef
to
bc171d7
Compare
This avoids having to re-check bounds every time we perform SAD, as the region knows its own size. It also may save 2 usize's being passed around during ME. To enforce this, we also remove the w and h parameters from everywhere.
This is part of a series of commits authored by @maj160 to improve performance of rav1e.