Skip to content

chore(local-testing): keep race panels error-free when no race is selected - #369

Merged
bkonick merged 1 commit into
mainfrom
chore/race-vars-no-race
Nov 1, 2026
Merged

bkonick merged 1 commit into
mainfrom
chore/race-vars-no-race

Conversation

@bkonick

@bkonick bkonick commented Nov 1, 2026

Copy link
Copy Markdown
Member

The Laps, Race Control, and Standings dashboards derive the hidden race_from / race_to variables from the selected race. When no race is selected — e.g. a fresh local stack with an empty races table — those queries returned no row, the variables were empty, and every panel filtering on time >= ${race_from}ms failed with invalid operation: time and *influxql.VarRef are not compatible.

Both queries now aggregate with max() so they always return one row: race_from falls back to 0 and race_to to now. With no race selected, the panels show "No data" instead of an error. For a selected race the values are unchanged, including now for a race with no end_time yet.

Verified against the local stack: the new and old SQL return identical values for a finished race and an open race, and every race-dependent panel across the three dashboards now queries without error when no race exists.

🤖 Generated with Claude Code

…ected

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one): 1
  • ai-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6c91022d-9280-42e0-8ff6-cf0510621ef6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@bkonick
bkonick merged commit 13512c0 into main Nov 1, 2026
7 checks passed
@bkonick
bkonick deleted the chore/race-vars-no-race branch November 1, 2026 15:01
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.

1 participant