-
Notifications
You must be signed in to change notification settings - Fork 0
Swerve #3
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
Open
Keller6738
wants to merge
41
commits into
master
Choose a base branch
from
swerve
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Swerve #3
Changes from 9 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
a6b3d13
feat: add copy constructor
Keller6738 103c779
feat: add copy constructor and constructor with default soft limits
Keller6738 dec3f49
feat: add an option to use only part of the limits
Keller6738 8aee716
feat: sort the swerve, make it abstract and simplify configuration
Keller6738 81b48ef
add: utils to simplify swerve configuration
Keller6738 c5310ea
feat: sort Odometry.java in a separate package
Keller6738 2b7594d
add: code example for swerve configuration
Keller6738 e7aefc8
fix: little fixes in the example code
Keller6738 a05f402
fix: getVelocityDistance method in ModulesHolder and little fixes in …
Keller6738 939dcb1
sort: SwerveMechanism and ModulesHolder classes
Keller6738 c900d2c
add: example initAutoBuilder method
Keller6738 3c6bd92
sort: SwerveModule class
Keller6738 7090b15
add: ModulesHolder docs
Keller6738 378c85e
add: SwerveMechanism docs
Keller6738 9496a1c
add: SwerveSubsystem example docs
Keller6738 d4392d5
add: SwerveAccUtils docs
Keller6738 cec9670
add: SwerveConfigurationUtils docs
Keller6738 1257ff0
add: SwerveModuleType docs and getters
Keller6738 ab118bc
add: SwerveSpecs docs and finished documentation of the swerve
Keller6738 f3a0fbf
feat: make circle extends Ellipse2d and add some methods
Keller6738 cd95d4d
feat: add constructor for a point and slope and getSlope method
Keller6738 ae6628b
feat: add some utils functions
Keller6738 ad29d50
feat: add an example for drive command with obstacle avoidance
Keller6738 76acae6
fix: delete swerve constants from the main Constants class
Keller6738 4d8d35d
fix: null pointer exception
Keller6738 b2cb2b2
feat: add a conversion factor from rotations to radians for the angle…
Keller6738 9c91c6b
fix: sort drive with obstacle avoidance method
Giladkeller 5bd1848
feat: add an option to create custom swerve type
Keller6738 246bf85
fix: fix the utility functions to match the new implementation
Keller6738 4de6907
fix: update the swerve example to match the new implementation
Keller6738 84109fb
feat: add more module types to SwerveModuleType
Keller6738 48a196b
made keller's code not extremely stupid and unobvious
YehudaRothstein 8accfec
refactor: streamline method signatures and improve variable naming in…
YehudaRothstein 0d1ceec
feat: switch from monologue to epilogue
Keller6738 b0ccbf7
feat: switch from monologue to epilogue
Keller6738 9a8fc60
feat: switch from monologue to epilogue
Keller6738 9ac40ac
fix: enum name
Keller6738 0fbdcfb
delete: unnecessary classes
Keller6738 eea2fc6
move: odometry package to control package
Keller6738 d26d24e
fix: add @NotLogged annotation to constants
Keller6738 6d7b86a
feat: switch from monologue to epilogue
Keller6738 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
...ava/frc/excalib/slam/mapper/Odometry.java → ...xcalib/slam/mapper/odometry/Odometry.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
5 changes: 5 additions & 0 deletions
5
src/main/java/frc/excalib/slam/mapper/odometry/VisionMeasurement.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| package frc.excalib.slam.mapper.odometry; | ||
|
|
||
| import edu.wpi.first.math.geometry.Pose2d; | ||
|
|
||
| public record VisionMeasurement(Pose2d estimatedRobotPose, double timestampSeconds) {} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.