-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add TuneG, TuneS, and TuneV #25
Comments
@jkleiber do you think it's okay to put these in wpi-interface even though technically they should be under control? They depend on WPI and I want to keep them in the right bucket if possible. Perhaps we could allow multiple submodules in wpi interface or something? |
@aidnem I think this is a perfectly acceptable use of wpi_interface because these commands are interfacing the generic controls Tunable with the WPILib |
I'm having some issues importing other packages while writing code in wpi interface. Perhaps we could look into this tomorrow as it should be an otherwise easy port. |
Now that I know how to use wpilib imports again I can finally finish this. |
Scope
Add the TuneG, TuneS, and TuneV classes from 2024 so that we can use Tunable more effectively.
The text was updated successfully, but these errors were encountered: