Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NET_HANDLER] Runtime status #267

Open
benliao1 opened this issue Jun 22, 2023 · 1 comment
Open

[NET_HANDLER] Runtime status #267

benliao1 opened this issue Jun 22, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request high-priority Issues that should be fixed as soon as possible.

Comments

@benliao1
Copy link
Collaborator

Currently, Runtime has no way of communicating to Dawn or Shepherd what the robot's status is. This information would be very useful for students (using Dawn) and staff (using Shepherd) to know, without having to call a member of Runtime over to ssh into the robot and open the shared memory UI.

This should be implemented using a new protobuf, defined in the protos repo, and should contain the following information:

  • Runtime version
  • Dawn connection status
  • Shepherd connection status
  • Battery voltage
  • Run mode

This message shall be sent on the following conditions:

  • When x number of seconds has passed since the last Runtime status message was sent to this client (we should determine an appropriate x ... probably 10 or 30 seconds?)
  • When Dawn or Shepherd connects or disconnects
  • When the run mode changes

The message shall be sent to all currently active clients, i.e. if Shepherd sends a run mode change to Runtime, Runtime should immediately send a Runtime status message to both Dawn and Shepherd confirming that it has indeed changed into the new run mode.

Resolving this issue will also resolve #208 and #209 .

@benliao1 benliao1 added enhancement New feature or request high-priority Issues that should be fixed as soon as possible. labels Jun 22, 2023
@BrandonWong14
Copy link
Contributor

We could also add whether to replace the battery or not to the list if things that need to be sent in runtime status.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request high-priority Issues that should be fixed as soon as possible.
Projects
None yet
Development

No branches or pull requests

6 participants