Skip to content
Snippets Groups Projects

Add clamping back to mv projection

Merged Add clamping back to mv projection
kkourin/dav1d:fix-projection-no-clamp into master
All threads resolved!
Merged Colin Lee requested to merge kkourin/dav1d:fix-projection-no-clamp into master
All threads resolved!

Commit aca57bf3 removed clamping from the projection calculation of motion vectors. The clamping here affects the motion vector selection process and is required by spec.

Example: test.ivf (crashes at frame 67 with HEAD)

Edited by Colin Lee

Merge request reports

Pipeline #18986 passed

Stage:
    Stage:
      Stage:

        Pipeline passed for 07261e8c on kkourin:fix-projection-no-clamp

        Test coverage 98.19% (-0.02%) from 1 job
        Loading
        Loading

        Activity

        Filter activity
        • Approvals
        • Assignees & reviewers
        • Comments (from bots)
        • Comments (from users)
        • Commits & branches
        • Edits
        • Labels
        • Lock status
        • Mentions
        • Merge request status
        • Tracking
      • Colin Lee changed the description

        changed the description

      • Colin Lee resolved all threads

        resolved all threads

      • Colin Lee added 1 commit

        added 1 commit

        • 673baf1c - Add clamping back to mv projection

        Compare with previous version

      • Colin Lee mentioned in merge request dav1d-test-data!71 (closed)

        mentioned in merge request dav1d-test-data!71 (closed)

      • Ronald S. Bultje
      • Colin Lee resolved all threads

        resolved all threads

      • Colin Lee added 2 commits

        added 2 commits

        • 80aa0c12 - Add clamping back to mv projection
        • 4f282f49 - Clean up fraction calculation

        Compare with previous version

      • Colin Lee added 2 commits

        added 2 commits

        • 2d4711c9 - Add clamping back to mv projection
        • 07261e8c - Clean up fraction calculation

        Compare with previous version

      • Ronald S. Bultje merged

        merged

      • Ronald S. Bultje mentioned in merge request dav1d-test-data!90 (merged)

        mentioned in merge request dav1d-test-data!90 (merged)

        • Please register or sign in to reply

        VideoLAN code repository instance