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

Move SwerveDrive default command to subsystem and remove our commands folder/package #14

Merged
merged 21 commits into from
Feb 1, 2025

Conversation

Ernie3
Copy link
Member

@Ernie3 Ernie3 commented Jan 31, 2025

  • Remove frc.robot.commands package since we now follow more modern practices
  • Move DriveSwerveWithXbox logic to the subsystem as a method to follow more modern practices
  • Create HIDUtils in an effort to decouple more logic
  • Create default commands for updating swerve odometry using vision instead of directly calling them in the subsystem
  • Give names to some commands where we inline create them
  • Update RevLib and PathPlanner libraries to latest
  • Utilize initSendable for SwerveDrive, SwerveModule, DemonLimelightCamera and DemonPhotonCamera

@Ernie3 Ernie3 merged commit 1cf74bf into main Feb 1, 2025
1 check passed
@Ernie3 Ernie3 deleted the swerve-default-cmd branch February 1, 2025 05:53
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