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.

Sign up for GitHub

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

Jump to bottom

Dashboard #17

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Dashboard #17

wants to merge 15 commits into from
+170 −21

Conversation

HenryLi-0
Copy link
Member

@HenryLi-0 HenryLi-0 commented Jan 19, 2025 •
edited
Loading

  • an operator dashboard (code in a separate repository) sends information to the robot over NetworkTables, which then listens to the data and has triggers for them
  • currently, there are triggers for the 12 branches,triggers for the 4 levels, and a trigger for processor (17 total)
  • works in simulation and should in real life (sending information to the robot has been tested and works!)
  • the code here just listens to network tables and does the triggers (dashboard GUI not included)
  • see the dashboard (which sends the data) over at https://github.com/HenryLi-0/Reefscape-2025-Dashboard

@HenryLi-0 HenryLi-0 requested review from Yxhej and kishan243 January 23, 2025 18:28
Unclesdad
Unclesdad previously approved these changes Jan 23, 2025
View reviewed changes
Copy link
Contributor

@Unclesdad Unclesdad left a comment

Choose a reason for hiding this comment

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

test

etangent
etangent previously approved these changes Jan 23, 2025
View reviewed changes
sigalrmp
sigalrmp previously approved these changes Jan 23, 2025
View reviewed changes
@sigalrmp sigalrmp dismissed stale reviews from Unclesdad, etangent, and themself January 23, 2025 20:14

test

@etangent etangent requested a review from a team as a code owner January 23, 2025 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Unclesdad Unclesdad Unclesdad left review comments

@etangent etangent etangent left review comments

@sigalrmp sigalrmp sigalrmp left review comments

@Yxhej Yxhej Awaiting requested review from Yxhej

@kishan243 kishan243 Awaiting requested review from kishan243

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@HenryLi-0 @sigalrmp @etangent @Unclesdad