Skip to content

Feat/grayson/risk UI#69

Open
grs2116 wants to merge 12 commits into
mainfrom
feat/grayson/risk-ui
Open

Feat/grayson/risk UI#69
grs2116 wants to merge 12 commits into
mainfrom
feat/grayson/risk-ui

Conversation

@grs2116
Copy link
Copy Markdown

@grs2116 grs2116 commented May 6, 2026

Description

This adds a Svelte overlay to display live telemetry over camera feed.

How Has This Been Tested?

This has been tested using docker and running natively. It includes a MQTT producer for data for viewing changes.

Checklist:

  • New functionality is documented in the necessary spots (i.e new functions documented in the header)
  • Unit tests cover any new functionality or edge cases that the PR was meant to resolve (if applicable)
  • The CI checks are passing
  • I reviewed my own code in the GitHub diff and am sure that each change is intentional

@grs2116 grs2116 requested a review from K3das May 15, 2026 16:58
Copy link
Copy Markdown
Contributor

@K3das K3das left a comment

Choose a reason for hiding this comment

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

Overall, looks great. Could you add a query string toggle for the synthetic data and a parameter for the MQTT address? Oh, and, can you run pnpm exec eslint --fix to fix linter issues (I'll add this check to CI at some point).

Regarding more nit-picky things, er, things that are entirely subjective; I think it would make a bit more sense to move the lat/long within the map's container to maintain a slightly better information hierarchy. In a similar way, the target values in the altitude and g-force sliders were a little unclear to me at first (I wasn't sure what they meant), so maybe changing the symbology of the set point can help.

Anyways, I'm not too bothered about testing it against real MQTT until I have a way to send fake data lol, which I will work on soon tm.

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