Skip to content

Driver Network Table #29

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

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

NotQuiteApex
Copy link
Contributor

From rationale commit:

Currently we use Shuffleboard for our dashboard on the driver station. The Driving network table contains all the data that we can push (which does not include from the SendableChooser and the Limelight camera feed) that the drive team needs to run a match. This table should be reserved for this purpose alone, as currently we simply push any data we may need during testing with SmartDashboard. As I understand, Shuffleboard allows us to use data from other tables on the same page as the ones specifically for it, and we can set up a nice view that will remain persistent between reboots of the robot and not be cluttered with data the drive team does not need.

Currently we use Shuffleboard for our dashboard on the driver station. The Driving network table contains all the data that we can push (which does not include from the SendableChooser and the Limelight camera feed) that the drive team needs to run a match. This table should be reserved for this purpose alone, as currently we simply push any data we may need during testing with SmartDashboard. As I understand, Shuffleboard allows us to use data from other tables on the same page as the ones specifically for it, and we can set up a nice view that will remain persistent between reboots of the robot and not be cluttered with data the drive team does not need.
@rsammelson
Copy link
Contributor

I approve, contingent on it working in testing

@PhyXTGears-1720 PhyXTGears-1720 changed the base branch from competition/bloomington to develop March 10, 2020 22:29
@boxofrox boxofrox changed the base branch from develop to release March 10, 2020 23:09
@boxofrox boxofrox changed the base branch from release to develop March 10, 2020 23:09
@NotQuiteApex
Copy link
Contributor Author

I'm going to reiterate what we discussed last night so we have it written down in case the subject comes up again.

We would rather use SmartDashboard only if possible, because it does not have memory leak problems like Shuffleboard does, and if something goes wrong with Shuffleboard at a competition we need to use SmartDashboard which cannot access other tables besides its own. Another thing to note is that we do not have the ability to send SendableChooser equivalents to other tables (yet?) and we need to learn how they operate and how we can replicate them if we want to switch to another Dashboard and table in NetworkTables.

As I understand we are also going to encapsulate any programming specific pushes to SmartDashboard under a macro so that we can recompile and not be sending data the drive team does not need.

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