Skip to content
Snippets Groups Projects

Add test for checking mv projection clamping

Closed Colin Lee requested to merge kkourin/dav1d-test-data:mv-projection-data into master
1 unresolved thread

This clip crashes on frame 67 if the projection is not clamped (ref: dav1d!1023 (merged)). Incorrect motion vector selections starting from frame 65 eventually leads to symbol overread on frame 67.

Concerns I have with merging this:

  1. At least on my machine, this test takes 10s to run, which is significantly longer than every other test. For comparison, the rest of the tests take 60s total for my machine to run. I'm not sure how to fabricate a smaller test case with the same issue, but that would probably be more ideal.
  2. This is a very short clip from a Japanese television show. I am not sure what is the policy regarding using test data sourced from copyrighted materials.

Merge request reports

Pipeline #19752 passed

Pipeline passed for 0b55434f on kkourin:mv-projection-data

Approval is optional

Closed by Ronald S. BultjeRonald S. Bultje 3 years ago (Feb 18, 2022 9:47pm UTC)

Merge details

  • The changes were not merged into master.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading