Skip to content

Add finish line drawing to race replay UI#94

Merged
IAmTomShaw merged 4 commits intoIAmTomShaw:mainfrom
viralcodex:track-finish-line-addition
Jan 10, 2026
Merged

Add finish line drawing to race replay UI#94
IAmTomShaw merged 4 commits intoIAmTomShaw:mainfrom
viralcodex:track-finish-line-addition

Conversation

@viralcodex
Copy link
Contributor

@viralcodex viralcodex commented Dec 20, 2025

Introduced a new draw_finish_line function in ui_components.py to render a checkered finish line on the track. Integrated this function into the F1RaceReplayWindow rendering flow in race_replay.py to visually indicate the race finish line.

image

@IAmTomShaw , this should add some visual cue for users to track the finish of the drivers on the track

Introduced a new draw_finish_line function in ui_components.py to render a checkered finish line on the track. Integrated this function into the F1RaceReplayWindow rendering flow in race_replay.py to visually indicate the race finish line.
@IAmTomShaw
Copy link
Owner

Are you able to add this to the Qualifying interface too? :)

@viralcodex
Copy link
Contributor Author

viralcodex commented Dec 22, 2025

doing it now sir 😄

Updated the draw_finish_line function to support both qualifying and race sessions by accepting a session_type parameter. Modified QualifyingReplay to use the new draw_finish_line logic and refactored track drawing to store inner and outer points as instance variables.
@viralcodex
Copy link
Contributor Author

hi @IAmTomShaw, extended to qualifying as well.

image

@viralcodex
Copy link
Contributor Author

Hi @IAmTomShaw, this is implemented and can be merged I guess 😄

@viralcodex
Copy link
Contributor Author

Hi @IAmTomShaw, any updates when you're free thanks 😊

@IAmTomShaw
Copy link
Owner

Sorry I missed this. Great work! :)

@IAmTomShaw IAmTomShaw merged commit a10bff6 into IAmTomShaw:main Jan 10, 2026
@viralcodex viralcodex deleted the track-finish-line-addition branch January 10, 2026 21:29
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.

2 participants