Skip to content
Snippets Groups Projects

Resolve "Add utility script for comparison of regression test data"

All threads resolved!

This MR adds a utility script for the visual comparison of old (truth) and new (test) trajectories for our regression tests.

3 kinds of Errors are detected:

  1. The person has no counterpart in the other file
  2. The person has a different first/last frame than the corresponding person in the other file
  3. The person differs from the other in one point with an euclidean distance of at least 1 pixel (currently not user-definable)

For the first case, you see the trajectory of the person who didn't find a match (yellow for truth; pink for test) and trajectories which could have matched, but didn't from test and truth. If you can see that there are a test and a truth trajectory drawn as well, the difference is probably that the person was double tracked in one but not in the other case.

For the second and third you just see the 2 trajectories (green: truth, blue: test).

You see each paring only once, i.e. if 2 people have different start and end frames, you still watch them once.

One goes to the next pairing by pressing n.

NOTE: You need to install opencv for python (used for undistortion)

Closes #216 (closed)

Edited by d.kilic

Merge request reports

Merged by d.kilicd.kilic 3 years ago (Sep 13, 2021 3:15pm UTC)

Loading

Pipeline #45174 passed

Pipeline passed for a0bf8cef on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • d.kilic added 1 commit

    added 1 commit

    Compare with previous version

  • Schrödter, Tobias resolved all threads

    resolved all threads

  • Schrödter, Tobias approved this merge request

    approved this merge request

  • d.kilic added 1 commit

    added 1 commit

    • a948499b - Fix pause in comparison script

    Compare with previous version

  • d.kilic resolved all threads

    resolved all threads

  • d.kilic enabled an automatic merge when the pipeline for a948499b succeeds

    enabled an automatic merge when the pipeline for a948499b succeeds

  • merged

  • d.kilic mentioned in commit a0bf8cef

    mentioned in commit a0bf8cef

  • Please register or sign in to reply
    Loading