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

Added match log viewing #164

Merged
merged 2 commits into from
Oct 20, 2023
Merged

Added match log viewing #164

merged 2 commits into from
Oct 20, 2023

Conversation

jolewine
Copy link
Contributor

This change adds a basic viewer to the data that is logged into the match log files. It is the start of work for issue #119 and more features are planned, but as this is my first contribution I wanted to make sure I put this forward for review if there are issues with my approach.

Things I have on my roadmap for the feature:

  • Elevator buttons to move to next/previous connection/disconnection event in the log
  • Tracking the time an indicator is in a bad state (similar to the count up timer in the match play screen)
  • Adding match graphs for individual team logs
  • Adding match graphs that will show multiple teams to look for field wide issues

@patfair
Copy link
Member

patfair commented Oct 20, 2023

@jolewine thanks, this is great! No problems with your approach; I am happy to see whatever FTAs/CSAs would find useful done here.

For this PR please just run go fmt and then the checks should pass and I'll merge it.

@patfair patfair merged commit ab0cabf into Team254:main Oct 20, 2023
1 check passed
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