Skip to content
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

New autons for comp #94

Merged
merged 6 commits into from
Jul 20, 2024
Merged

Conversation

Rand0mAsianKid
Copy link
Contributor

Please merge the sunday-code branch into master as quickly as possible once everything is finished. I want to merge my autons into master that way I can use it during matches.

The autons that I created and that should be used at comp are:

Center-Auto Ruiner
Left-Auto Ruiner
Right-Auto Ruiner

Center Field Limelight(No Preload)
Center Field Limelight

Center Limelight 4 Piece
Left Limelight 4 Piece
Right Limelight 4 Piece

Center Limelight 1 Piece
Left Limelight 1 Piece
Right Limelight 1 Piece

Backup-Left
Backup-Right
Backup-Center

Center-Forward
Left-Forward
Right-Forward

@Rand0mAsianKid Rand0mAsianKid requested a review from a team as a code owner July 20, 2024 08:28

/** Add your docs here. */
public class AlignDrivetrain extends ProxyCommand {
static double blueSpeakerX = 0.14;
Copy link
Member

Choose a reason for hiding this comment

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

This is not used so the math can't be right.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah we aren't using that command any more

Copy link
Contributor

@ProfessorAtomicManiac ProfessorAtomicManiac left a comment

Choose a reason for hiding this comment

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

if we aren't using alignDrivetrain.java then these changes shouldn't downgrade the robot in any capacity. (did not check autos very carefully tho)

@ProfessorAtomicManiac ProfessorAtomicManiac merged commit 218bf3e into sunday-code Jul 20, 2024
1 check passed
@ProfessorAtomicManiac ProfessorAtomicManiac deleted the New-Autons-for-Comp branch July 20, 2024 15:51
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.

3 participants