feat: add SPEED_AND_DISTANCE combo and ALL orchestrator (PR 5)#64
Draft
AlexBodner wants to merge 2 commits into
Draft
feat: add SPEED_AND_DISTANCE combo and ALL orchestrator (PR 5)#64AlexBodner wants to merge 2 commits into
AlexBodner wants to merge 2 commits into
Conversation
AlexBodner
force-pushed
the
feat/soccer-pr5-combo
branch
from
b30b142 to
b39d4bf
Compare
Extract shared trace renderer from distance.py, add dim/spotlight helpers, run_all batching over a single VideoTrackingSession, and wire new modes in main. Co-authored-by: Cursor <cursoragent@cursor.com>
DISTANCE shows direction dots and distance chips only; SPEED_AND_DISTANCE keeps m/s badges plus distance. Co-authored-by: Cursor <cursoragent@cursor.com>
AlexBodner
force-pushed
the
feat/soccer-pr5-combo
branch
from
b39d4bf to
1d85390
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
SPEED_AND_DISTANCEmode via a thin wrapper around PR4's_render_speed_distance_traces(follow-all or spotlight via--track-id).ALLmode /run_all.py: oneVideoTrackingSessiondrives five outputs (direction, speed, distance, speed-distance-all, speed-distance-single).only_tidfiltering land in PR4.Stack: PR 1 → PR 2 → PR 3 → PR 4 (base) → this PR
Demo
SPEED_AND_DISTANCE— follow-all: direction + m/s + distance chips (no end-card; re-rendered preview):SPEED_AND_DISTANCE— spotlight (--track-id):All demo clips ·
ALLmode writes the same five renders in one tracking pass.Reuse design
speed_and_distance.py— ~13-line wrapper calling_render_speed_distance_traces(no end-card).run_all.py— shared session orchestrator; spotlight auto-pick = maxdistance_mwhen--track-idomitted.Scope
Mode.SPEED_AND_DISTANCE,Mode.ALL,--track-idinmain.pyNot included: GK goal-distance locks, goal shading (PR6), ball/possession/pass (PR7+).
Test plan
python main.py ... --mode SPEED_AND_DISTANCE --max-frames 90 --tracker bytetrackpython main.py ... --mode SPEED_AND_DISTANCE --track-id 7 ...python main.py ... --mode ALL --target_video_path data/renders/2e57b9_0.mp4 ...→ five suffixed outputs