Skip to content

added drive command#99

Open
JAMthepersonj wants to merge 5 commits intomainfrom
JW-added-auto-drive-command
Open

added drive command#99
JAMthepersonj wants to merge 5 commits intomainfrom
JW-added-auto-drive-command

Conversation

@JAMthepersonj
Copy link
Contributor

No description provided.

}

public void putShuffleboardCommands() {
SmartDashboard.putData(
Copy link
Contributor

Choose a reason for hiding this comment

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

move to if (DEBUG)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@Override
public void execute() {
switch (dir){
case BACKWARD -> drivebase.drive(new Translation2d(-1,0),0, true);
Copy link
Contributor

Choose a reason for hiding this comment

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

true is field relative. We need to be robot relative. Need to test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

private final double time;
private final DriveDirection dir;
private Timer timer;
public DriveSwerve(SwerveSubsystem drivebase, DriveDirection dir, double time) {
Copy link
Contributor

Choose a reason for hiding this comment

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

can we add speed to the constructor in m/sec?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not fixed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants