Fix jagged track map and Docker PORT handling - #102
Merged
Merged
Conversation
Track outline (#101): the outline was drawn from the session's fastest lap, but when a session's position feed repeats coordinates instead of updating them, that lap has too few distinct points to trace the circuit. The 2026 Hungarian race gave 26 distinct points against ~300 in qualifying, rendering a 26-sided polygon. Every lap in that session is equally affected, so the reference lap is now chosen by distinct position points rather than lap time, falling back to another session from the same weekend. Both the outline and the replay coordinate normalisation go through the same helper so driver dots stay aligned with the track. Docker PORT: PORT set the host side of the compose mapping while the container side was hardcoded to 8000, and env_file also passed PORT into the container. Setting it published one port and listened on another. Compose now maps the same port on both sides, with PORT set explicitly so a shell-provided value reaches the container too. Also adds backend/.fastf1-cache to .dockerignore; the existing entry pointed at backend/data/fastf1-cache, so a local precompute put 3.2GB into the build context. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Track outline (#101): the outline was drawn from the session's fastest lap, but when a session's position feed repeats coordinates instead of updating them, that lap has too few distinct points to trace the circuit. The 2026 Hungarian race gave 26 distinct points against ~300 in qualifying, rendering a 26-sided polygon. Every lap in that session is equally affected, so the reference lap is now chosen by distinct position points rather than lap time, falling back to another session from the same weekend. Both the outline and the replay coordinate normalisation go through the same helper so driver dots stay aligned with the track.
Docker PORT: PORT set the host side of the compose mapping while the container side was hardcoded to 8000, and env_file also passed PORT into the container. Setting it published one port and listened on another. Compose now maps the same port on both sides, with PORT set explicitly so a shell-provided value reaches the container too.
Also adds backend/.fastf1-cache to .dockerignore; the existing entry pointed at backend/data/fastf1-cache, so a local precompute put 3.2GB into the build context.