Skip to content

CIRC 0.5 updates: stale indicators, disable go, and reset all#49

Merged
codeflight1 merged 2 commits into
mainfrom
circ0.5
Jul 14, 2026
Merged

CIRC 0.5 updates: stale indicators, disable go, and reset all#49
codeflight1 merged 2 commits into
mainfrom
circ0.5

Conversation

@codeflight1

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the dashboard’s operational UX for CIRC 0.5 by adding “stale” status indicators (to surface missing telemetry), preventing arm motion commands when the arm isn’t idle, and adding a one-click “Reset All” action for motor error resets.

Changes:

  • Add stale-status countdown/indicator to the node status panel.
  • Add “Reset All” and stale-status handling to the motor status panel.
  • Disable the Arm “Go” action unless a pose is selected and the arm is in an Idle state.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/components/panels/NodeStatusPanel.tsx Adds a global stale timer and UI indicator for node status updates.
src/components/panels/MotorStatusPanel.tsx Adds “Reset All” plus per-motor stale countdown and updated LED/error display logic.
src/components/panels/ArmControlPanel.tsx Prevents “Go” command unless the arm is Idle.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/panels/NodeStatusPanel.tsx Outdated
Comment thread src/components/panels/MotorStatusPanel.tsx
Comment thread src/components/panels/MotorStatusPanel.tsx Outdated
Comment thread src/components/panels/ArmControlPanel.tsx Outdated
@ConnorNeed

Copy link
Copy Markdown
Member

I'm pretty rough at reviewing typescript. I tagged copilot to take a look. Double check the things it came up with. The busy timer is the one that scares me the most.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@codeflight1

Copy link
Copy Markdown
Contributor Author

Looks like it found what I couldn't, which was a way to do them all at once, so I feel better about it now

@codeflight1 codeflight1 merged commit 37a6ef0 into main Jul 14, 2026
1 of 2 checks passed
@codeflight1 codeflight1 deleted the circ0.5 branch July 14, 2026 03:13
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.

3 participants