-
Notifications
You must be signed in to change notification settings - Fork 7
intake #18
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
base: main
Are you sure you want to change the base?
intake #18
Conversation
Incomplete; missing support in Machinery and Binder, still needs to read the angle of the motor.
| public class IntakeExtend extends SubsystemBase { | ||
| private final BareMotor m_motor; | ||
|
|
||
| public Intake(LoggerFactory parent) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change the constructor name to IntakeExtend here.
| final AprilTagRobotLocalizer m_localizer; | ||
| final SwerveDriveSubsystem m_drive; | ||
| final Beeper m_beeper; | ||
| final Intake m_intake; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add IntakeExtend declaration here
| // | ||
|
|
||
| // Subsystem initializers go here. | ||
| m_intake = new Intake(driveLog); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add IntakeExtend initialization here
| /// SUBSYSTEMS | ||
| /// | ||
| whileTrue(driver::x, m_machinery.m_intake.intake()); | ||
| whileTrue(driver::x, m_machinery.m_intake.intake()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use driver::y and bind to intake extend here
|
|
||
| m_machinery.m_intake.setDefaultCommand( | ||
| m_machinery.m_intake.stop()); | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add intake extend default command here
| import edu.wpi.first.wpilibj2.command.Command; | ||
| import edu.wpi.first.wpilibj2.command.SubsystemBase; | ||
|
|
||
| public class IntakeExtend extends SubsystemBase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
take a look at team100.frc2025.Climber for an example to follow for positional control here. it's ok to just cut-and-paste some of it (look at the "default" part in the constructor) and then we can discuss in this PR.
No description provided.