Skip to content

Conversation

@truher
Copy link
Member

@truher truher commented Jan 29, 2026

No description provided.

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) {
Copy link
Member Author

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;
Copy link
Member Author

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);
Copy link
Member Author

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());
Copy link
Member Author

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());

Copy link
Member Author

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 {
Copy link
Member Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants