Skip to content

telemetry/track map using csv#220

Open
celinayang13 wants to merge 14 commits into
mainfrom
telemtry-track-mapper
Open

telemetry/track map using csv#220
celinayang13 wants to merge 14 commits into
mainfrom
telemtry-track-mapper

Conversation

@celinayang13
Copy link
Copy Markdown
Contributor

No description provided.

@celinayang13 celinayang13 requested a review from a team as a code owner April 18, 2026 05:09
@celinayang13 celinayang13 requested review from angelasrsh and samuelLi05 and removed request for a team April 18, 2026 05:09
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lhre-2026 Ready Ready Preview, Comment Apr 18, 2026 7:43am

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces significant updates to the telemetry database schema and the track mapping tool. Key changes include the addition of lookup tables (LUTs) for drivers, cars, and locations, enhanced validation for drive day creation, and the integration of a new Track Mapper feature in the live viewer. The Track Mapper now supports CSV uploads and improved path smoothing. Feedback focuses on critical schema mismatches in the Prisma model, specifically regarding primary keys and relations in the classifier and track_point models. Additionally, improvements were suggested for the track completion logic in the Python processor to prevent premature triggers, and for the efficiency of finding the last valid track point in the frontend components.

Comment thread telemtry/analysis/database/viewer_tool/prisma/orion.prisma
Comment thread telemtry/analysis/database/viewer_tool/prisma/orion.prisma
Comment thread telemtry/analysis/database/viewer_tool/prisma/orion.prisma
Comment thread telemtry/stack/processors/track_mapper/main.py Outdated
Comment thread telemtry/analysis/database/viewer_tool/src/components/TrackMapper.tsx Outdated
Comment thread telemtry/analysis/database/viewer_tool/src/components/TrackMapper.tsx Outdated
celinayang13 and others added 2 commits April 18, 2026 00:23
…per.tsx

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…per.tsx

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants