Conversation
This reverts commit 9435410.
This reverts commit 7469dc0.
…gic, added ability to limit flywheel speed for testing
Resolved merge conflict in AlignUtil.java by accepting incoming changes from aim! branch. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
swerve uses motion magic + driver keybinds
- Fixed shooter position calculation by properly rotating offset by robot heading - Removed incorrect Transform2d rotation that was affecting target angle - Converted static methods to instance methods to prevent null reference issues - Updated RobotContainer to use new createAimCommand() method - Added resetToStartingPosition() method to SwerveSubsystem for testing - Bound Options button to reset pose to red hub neutral zone (10.4, 4.0) - Fixed deprecated getPositionError() call in RotateToAngleCommand - Added options() button accessor to PS5DriveController NEED TO DO: Fix pigion heading issue
…dont need a negitive pid, also offset turn by 90* because we dont want to allign to front rather want to do right side, also fixed the keybind issue where it counted the trigger r2 being pressed as a cancel
There was a problem hiding this comment.
Line 18: shootoffset should probably be in shooter constants
There was a problem hiding this comment.
Earlier comment extends to this, Cancelcondition not implemented to beware
|
this should [probably] be merged into premergeofdoom |
There was a problem hiding this comment.
Looks good, implement Cancelcondition at some point
There was a problem hiding this comment.
So you guys don't want to reset the hood angle periodically? That or this is clubbing a feature
There was a problem hiding this comment.
Printlns + shouldn't this be a singleton?
There was a problem hiding this comment.
Alot of formatting changes assumedely as per GRT conventions.
Reminding myself to take a second look but why is acceleration limiting being implemented now?
4 participants
No description provided.