Skip to content

92 event comparison over time#104

Merged
samuelbaldwin05 merged 12 commits intomainfrom
92-event-comparison-over-time
Apr 3, 2026
Merged

92 event comparison over time#104
samuelbaldwin05 merged 12 commits intomainfrom
92-event-comparison-over-time

Conversation

@dboghra
Copy link
Copy Markdown
Collaborator

@dboghra dboghra commented Mar 24, 2026

Summary

Closes # 92

What changed:
Added an event time history visualization that lets coaches see an athlete's times for a single event across multiple completions. in trends section of the toggle on the athletes' profile. Includes a filter bar with an event type dropdown and an amount picker (last N runs or date range) or input a range, and a line chart showing run number vs total time in seconds with a reference line for the best time.

Why:
Coaches needed a way to track performance trends for a specific event over time, rather than viewing individual run analyses in isolation.

Technical decisions worth noting:
Any non-obvious choices and their rationale.
-Elapsed_ms is read directly from the run table since total run time is already stored there
-The chart only fetches data on explicit Apply click rather than reactively, to avoid unnecessary requests while the coach is still making selections

AI disclosure: Note what was AI-assisted vs. hand-written.
Feature architecture and file structure were AI-assisted via Claude. All code was reviewed, debugged, and integrated by hand, including resolving the useState placement bug, the elapsed_ms null handling, and the routing setup.


Screenshot

image

Notes

@bamarler bamarler temporarily deployed to supabase-testing March 25, 2026 03:57 — with GitHub Actions Inactive
@bamarler bamarler temporarily deployed to supabase-testing March 25, 2026 03:57 — with GitHub Actions Inactive
@bamarler bamarler linked an issue Mar 25, 2026 that may be closed by this pull request
@bamarler bamarler temporarily deployed to supabase-testing March 26, 2026 20:59 — with GitHub Actions Inactive
@bamarler bamarler temporarily deployed to supabase-testing March 26, 2026 20:59 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 28, 2026 03:38 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 28, 2026 03:38 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 31, 2026 00:15 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 31, 2026 00:15 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 31, 2026 03:16 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing March 31, 2026 03:16 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:10 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:10 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:14 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:14 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:16 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:16 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:21 — with GitHub Actions Inactive
@dboghra dboghra temporarily deployed to supabase-testing April 3, 2026 15:21 — with GitHub Actions Inactive
@samuelbaldwin05 samuelbaldwin05 merged commit 7880527 into main Apr 3, 2026
11 checks passed
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.

Event Comparison over Time

3 participants