Skip to content

feat: add DISTANCE mode with traces and leaderboard (PR 4) - #63

Open
AlexBodner wants to merge 5 commits into
feat/soccer-pr3-speedfrom
feat/soccer-pr4-distance
Open

feat: add DISTANCE mode with traces and leaderboard (PR 4)#63
AlexBodner wants to merge 5 commits into
feat/soccer-pr3-speedfrom
feat/soccer-pr4-distance

Conversation

@AlexBodner

@AlexBodner AlexBodner commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds DISTANCE mode: direction dots + cumulative meters + trace minimap + distance leaderboard end-card (no m/s badges).
  • Distance via gated H on VideoTrackingSession.tracks.
  • Shared _render_speed_distance_traces(..., show_speed=) for PR5 reuse.

Demo

DISTANCE on 08fd33_0.mp4 (full-clip render, truncated preview):

All demos

Scope

In: distance.py, kinematics distance APIs, trace/end-card annotators, Mode.DISTANCE

Out: combo/ALL wiring (PR5), goal shading draw (PR6)

Test plan

  • python main.py ... --mode DISTANCE --tracker bytetrack
  • Chips + traces + end-card; SPEED/DIRECTION unchanged

AlexBodner and others added 5 commits July 21, 2026 14:04
Integrate distance via gated homography on VideoTrackingSession.tracks while reusing gap-filled speed badges and minimap traces from PR3.

Co-authored-by: Cursor <cursoragent@cursor.com>
Share _render_speed_distance_traces in distance.py with focus_tid and append_end_card hooks, plus spotlight helpers and only_tid speed filtering, so PR5 adds wiring only.

Co-authored-by: Cursor <cursoragent@cursor.com>
Shared renderer accepts show_speed; SPEED_AND_DISTANCE wiring in PR5 uses show_speed=True.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@AlexBodner
AlexBodner force-pushed the feat/soccer-pr4-distance branch from 6792f90 to b4f0709 Compare July 21, 2026 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant