Skip to content

BallChucker9000 + Limelight Command Based #62

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 24 commits into
base: Command-Based
Choose a base branch
from

Conversation

cadenmr
Copy link
Contributor

@cadenmr cadenmr commented Mar 14, 2020

Initial (& basic) implementation.

Ready for testing:

  • NetworkTables
  • Auto Rotate
  • Manual Rotate
  • Auto Flywheel
  • Flywheel Spin-Up
  • Indexer

Closes #55 and Closes #15

@cadenmr cadenmr requested a review from TaeSeanDo March 14, 2020 01:55
@cadenmr cadenmr self-assigned this Mar 14, 2020
@cadenmr cadenmr added the Command based This is specific to the command based system label Mar 14, 2020
@cadenmr cadenmr added this to the MVP Command-Based milestone Mar 14, 2020

@Override
public void initialize() {

Copy link
Contributor

Choose a reason for hiding this comment

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

Remember to fill this in. We probably want 2 commands: One with limelight input, the other that just takes a DoubleSupplier or double speed

@@ -25,14 +29,16 @@

private final ExampleCommand m_autoCommand = new ExampleCommand(m_exampleSubsystem);


private final BallChucker9000 BallChucker9000Subsystem;

/**
* The container for the robot. Contains subsystems, OI devices, and commands.
*/
public RobotContainer() {
// Configure the button bindings
configureButtonBindings();
Copy link
Contributor

Choose a reason for hiding this comment

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

By the way, here's the buttons.
manipulator -- left bumper: manual rotator left.
right bumper: manual rotator right.
Right joystick -- trigger: shoot
Button 3: indexer forward
Button 2: indexer backward

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Command based This is specific to the command based system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants