Closed greenden007 closed 1 year ago
There may be other places we need the pigeon and don't want the drivetrain. Let's keep them separate.
And when we copy over code from year to year, it keeps the code organized into proper areas. If I need a pigeon on the dashboard for testing purposes, I don't need a drivetrain.
On Wednesday, February 8, 2023 at 05:09:15 PM CST, Rohan C ***@***.***> wrote:
@greenden007 commented on this pull request.
In src/main/java/team/gif/robot/UiSmartDashboard.java:
public class UiSmartDashboard { +
- public SendableChooser
autoModeChooser = new SendableChooser<>(); - public SendableChooser
delayChooser = new SendableChooser<>(); - public UiSmartDashboard() {
- ShuffleboardTab tab = Shuffleboard.getTab("SmartDashboard"); // Gets a reference to the shuffleboard tab
- tab.add("BotHead", (x) -> {
- x.setSmartDashboardType("Gyro");
- x.addDoubleProperty("Value", () -> Robot.swervetrain.getHeading().getDegrees(), null);
That's the function I wrote to get the pigeon heading and I didn't want to make the pigeon public
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because your review was requested.Message ID: @.***>
Looks good overall. Let's test it tonight and it works on both bots, we can merge it to main.
Add auto delay